Re: [PATCH] Add section headings to index types doc

Поиск
Список
Период
Сортировка
От Jürgen Purtz
Тема Re: [PATCH] Add section headings to index types doc
Дата
Msg-id 159706393772.790.11301900224459454375.pgcf@coridan.postgresql.org
обсуждение исходный текст
Ответ на Re: [PATCH] Add section headings to index types doc  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: [PATCH] Add section headings to index types doc  (Michael Paquier <michael@paquier.xyz>)
Список 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 по дате отправления:

Предыдущее
От: Amit Langote
Дата:
Сообщение: Re: problem with RETURNING and update row movement
Следующее
От: Pavel Borisov
Дата:
Сообщение: Re: [PATCH] Covering SPGiST index