Re: [PATCH] Improve documentation of REINDEX options

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Josef Šimánek <josef(dot)simanek(at)gmail(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: [PATCH] Improve documentation of REINDEX options
Date: 2019-12-17 05:36:31
Message-ID: 20191217053631.GL2344@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Dec 13, 2019 at 10:28:33AM +0100, Josef Šimánek wrote:
> I have prepared patch to improve documentation for REINDEX. It
> should be more inline with another documentation pages.
>
> You can see the change applied in attached file. Patch can be found at
> https://github.com/simi/postgres/pull/3 (diff -
> https://github.com/simi/postgres/pull/3.diff, patch -
> https://github.com/simi/postgres/pull/3.patch).

Please, always attach your patches to emails sent on this mailing
list. If for a reason or another, the data located to with external
link is lost (imagine for example that your github account is gone or
that github is reduced to ashes), then such patches would be lost, and
anybody looking at this email 10 years from now would not know what
you have been writing about here. I am attaching it here for the
archive's sake.

+<phrase>where <replaceable class="parameter">option</replaceable> can
be:</phrase>
+
+ VERBOSE
Why not... We did that in the docs of ANALYZE for v11 when
introducing the parenthesized grammar flavor for the options
available.

- Rebuild all the indexes on the table <literal>my_table</literal>:
+ Rebuild all the indexes on the table <literal>my_table</literal>
with progress report per index:

<programlisting>
-REINDEX TABLE my_table;
+REINDEX (VERBOSE) TABLE my_table;
Not sure if this part brings much to the reader though. It is not
like the command description of REINDEX is complicated with dozens
of option choices.
--
Michael

Attachment Content-Type Size
reindex-doc.patch text/x-diff 1.4 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Suraj Kharage 2019-12-17 05:54:46 Re: backup manifests
Previous Message Michael Paquier 2019-12-17 05:19:13 Re: PATCH: standby crashed when replay block which truncated in standby but failed to truncate in master node