suggest about bpchar data type in the document

Started by yanliang leiover 3 years ago7 messagesdocs
Jump to latest
#1yanliang lei
msdnchina@163.com

I hope that:

in the https://www.postgresql.org/docs/14/datatype-characterin.html ,It needs to be written clearly “bpchar” data type,

and in the above url, Only mentioned “blank padded” ,If someone does not know this knowledge point(blank padded =bp),he/she will make a detour in the study of character datatype。

so,To make the document more friendly, I suggest that: in the https://www.postgresql.org/docs/14/datatype-characterin.htm ,It needs to be written clearly “bpchar” data type。

thanks a lot!

#2Laurenz Albe
laurenz.albe@cybertec.at
In reply to: yanliang lei (#1)
Re: suggest about bpchar data type in the document

On Wed, 2022-09-14 at 17:41 +0800, yanliang lei wrote:

I hope that:

 in the  https://www.postgresql.org/docs/14/datatype-characterin.html ,It needs to be written clearly  “bpchar” data type,

and in the above url, Only mentioned “blank padded” ,If someone does not know  this knowledge point(blank padded =bp),he/she  will make a detour in the study of  character datatype。

so,To make the document more friendly, I suggest that: in the  https://www.postgresql.org/docs/14/datatype-characterin.htm  ,It needs to be written clearly  “bpchar” data type。

+1

I think that adding "bpchar(n)" in table 8.4 would be a good thing.

Yours,
Laurenz Albe

#3Laurenz Albe
laurenz.albe@cybertec.at
In reply to: Laurenz Albe (#2)
Re: suggest about bpchar data type in the document

On Wed, 2022-09-14 at 18:48 +0200, Laurenz Albe wrote:

On Wed, 2022-09-14 at 17:41 +0800, yanliang lei wrote:

I hope that:

 in the  https://www.postgresql.org/docs/14/datatype-characterin.html ,It needs to be written clearly  “bpchar” data type,

and in the above url, Only mentioned “blank padded” ,If someone does not know  this knowledge point
(blank padded =bp),he/she  will make a detour in the study of  character datatype。

so,To make the document more friendly, I suggest that: in the  https://www.postgresql.org/docs/14/datatype-characterin.htm
It needs to be written clearly  “bpchar” data type。

+1

I think that adding "bpchar(n)" in table 8.4 would be a good thing.

Here is a patch for that.

Yours,
Laurenz Albe

Attachments:

0001-Document-bpchar-as-character-data-type.patchtext/x-patch; charset=UTF-8; name=0001-Document-bpchar-as-character-data-type.patchDownload+1-2
#4Tom Lane
tgl@sss.pgh.pa.us
In reply to: Laurenz Albe (#3)
Re: suggest about bpchar data type in the document

Laurenz Albe <laurenz.albe@cybertec.at> writes:

I think that adding "bpchar(n)" in table 8.4 would be a good thing.

Here is a patch for that.

There would need to be some mention of it in the paras below, too.

regards, tom lane

#5Laurenz Albe
laurenz.albe@cybertec.at
In reply to: Tom Lane (#4)
Re: suggest about bpchar data type in the document

On Wed, 2022-09-28 at 10:48 -0400, Tom Lane wrote:

Laurenz Albe <laurenz.albe@cybertec.at> writes:

I think that adding "bpchar(n)" in table 8.4 would be a good thing.

Here is a patch for that.

There would need to be some mention of it in the paras below, too.

True. I think the attached should be sufficient.

Yours,
Laurenz Albe

Attachments:

0001-Document-bpchar-as-character-data-type.V2.patchtext/x-patch; charset=UTF-8; name=0001-Document-bpchar-as-character-data-type.V2.patchDownload+6-6
#6Tom Lane
tgl@sss.pgh.pa.us
In reply to: Laurenz Albe (#5)
Re: suggest about bpchar data type in the document

Laurenz Albe <laurenz.albe@cybertec.at> writes:

On Wed, 2022-09-28 at 10:48 -0400, Tom Lane wrote:

There would need to be some mention of it in the paras below, too.

True. I think the attached should be sufficient.

Pushed. I failed to resist the temptation to do some further cleanup
in the same area.

regards, tom lane

#7Laurenz Albe
laurenz.albe@cybertec.at
In reply to: Tom Lane (#6)
Re: suggest about bpchar data type in the document

On Wed, 2022-09-28 at 12:32 -0400, Tom Lane wrote:

Laurenz Albe <laurenz.albe@cybertec.at> writes:

On Wed, 2022-09-28 at 10:48 -0400, Tom Lane wrote:

There would need to be some mention of it in the paras below, too.

True.  I think the attached should be sufficient.

Pushed.  I failed to resist the temptation to do some further cleanup
in the same area.

Thanks for your diligent failure, as always.

Yours,
Laurenz Albe