Re: [PATCH] Add section headings to index types doc
Jürgen Purtz <juergen@purtz.de>
From: Jürgen Purtz <juergen@purtz.de>
To: pgsql-hackers@lists.postgresql.org
Cc: Dagfinn Ilmari Mannsåker <ilmari@ilmari.org>
Date: 2020-08-10T12:52:17Z
Lists: 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
Commits
-
Doc: minor improvements for section 11.2 "Index Types".
- 85b4ba73423b 14.0 landed