Re: "box" type description
От | Bruce Momjian |
---|---|
Тема | Re: "box" type description |
Дата | |
Msg-id | ZUJwYXGeuVkWzkDB@momjian.us обсуждение исходный текст |
Ответ на | Re: "box" type description (Christoph Berg <myon@debian.org>) |
Ответы |
Re: "box" type description
|
Список | pgsql-hackers |
On Wed, Mar 31, 2021 at 01:43:47PM +0200, Christoph Berg wrote: > Re: Kyotaro Horiguchi > > Returing to the description of pg_types, it should be changed like > > this following the discussion here. > > > > - pg_catalog | box | geometric box '(lower left,upper right)' > > + pg_catalog | box | geometric box 'lower left,upper right' > > > > But I find it hard to read. I fixed it instead as the following in the > > attached. However, it might rather be better not changing it.. > > > > + pg_catalog | box | geometric box 'pt-lower-left,pt-upper-right' > > I like that because it points to the "point" syntax so users can > figure out how to spell a box. I liked Horiguchi-san's patch from 2021, but once I started looking further, I found a number of improvements that can be made in the \dTS output beyond Horiguchi-san's changes: * boolean outputs 't'/'f', not 'true'/'false' * Added "format" ... for output * tid output format was at the start, not the end * I didn't add space between point x,y because the output has no space * I spelled out "point" instead of "pt" * "line" has two very different input formats so I listed both (more different than others like boolean) * I didn't see the need to say "datatype" for LSN and UUID * I improved the txid_snapshot description * There was no description for table_am_handler Patch attached. -- Bruce Momjian <bruce@momjian.us> https://momjian.us EDB https://enterprisedb.com Only you can decide what is important to you.
Вложения
В списке pgsql-hackers по дате отправления: