Re: [PATCH] Add section headings to index types doc - Mailing list pgsql-hackers

From Jürgen Purtz
Subject Re: [PATCH] Add section headings to index types doc
Date
Msg-id 159706393772.790.11301900224459454375.pgcf@coridan.postgresql.org
Whole thread Raw
In response to Re: [PATCH] Add section headings to index types doc  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [PATCH] Add section headings to index types doc  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers
The following review has been posted through the commitfest application:
make installcheck-world:  not tested
Implements feature:       not tested
Spec compliant:           not tested
Documentation:            tested, passed

a) I'm wondering if we should apply one more change to this page. The line-by-line listing of operators occupies much
space.What do you think about an inline list (in a separate line) of the operators?
 

old source:
    <simplelist>
<member><literal><</literal></member>
<member><literal><=</literal></member>
<member><literal>=</literal></member>
<member><literal>>=</literal></member>
<member><literal>></literal></member>
    </simplelist>

new source:
   <synopsis><   <=   =   >=   ></synopsis>
It looks nice in HTML as well as in PDF.

b) I'm in favor of the indentation of all affected lines as it is done in the patch.

The new status of this patch is: Waiting on Author

pgsql-hackers by date:

Previous
From: Amit Langote
Date:
Subject: Re: problem with RETURNING and update row movement
Next
From: Pavel Borisov
Date:
Subject: Re: [PATCH] Covering SPGiST index