| From: | Michael Paquier <michael(at)paquier(dot)xyz> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Bruce Momjian <bruce(at)momjian(dot)us>, osdba <mailtch(at)163(dot)com>, pgsql-docs(at)postgresql(dot)org, jkatz(at)postgresql(dot)org |
| Subject: | Re: Document "59.2. Built-in Operator Classes" have a clerical error? |
| Date: | 2020-08-27 06:33:22 |
| Message-ID: | 20200827063126.GO2017@paquier.xyz |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-docs |
On Wed, Aug 26, 2020 at 07:19:09PM -0400, Tom Lane wrote:
> With one eye on the PDF width issue, I propose that we not draw
> the distinction, but just list all the relevant operators for each
> opclass (its native ones, plus the applicable "loose" operators).
> Then we only need two columns, opclass and operators.
Indeed, removing the types makes sense if we list them with the
operators. I have been looking at your suggestions, and adding a
space before the first parenthesis where the types are listed sounds
good to me, but I am not sure that it is a good idea to add spaces
between each type. Looking at the pdf produced, I think that we
should also drop entirely colspec for the BRIN table as it gets much
small in width once the data type column is removed. I also looked at
rowsep for the PDF, and I tend to prefer the version where we separate
each cell for the operator, as a matter of readability.
This leads me to the updated version attached. BRIN has 29 different
opclasses, visibly.
--
Michael
| Attachment | Content-Type | Size |
|---|---|---|
| doc-builtin-ops-v2.patch | text/x-diff | 46.6 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | PG Doc comments form | 2020-08-27 12:41:31 | PG_Basebackup not compatible with the version 12.3 |
| Previous Message | Bruce Momjian | 2020-08-27 02:38:30 | Re: Add comma after e.g. and i.e.? |