From: | Michael Paquier <michael(at)paquier(dot)xyz> |
---|---|
To: | Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr> |
Cc: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Why does pg_checksums -r not have a long option? |
Date: | 2019-05-27 08:33:30 |
Message-ID: | 20190527083330.GC25901@paquier.xyz |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, May 27, 2019 at 08:32:21AM +0200, Fabien COELHO wrote:
> I've used both -f & --filenode in the test to check that the renaming was
> ok. I have reordered the options in the documentation so that they appear in
> alphabetical order, as for some reason --progress was out of it.
No objection to clean up this at the same time.
+ <varlistentry>
+ <term><option>-f <replaceable>filenode</replaceable></option></term>
+ <term><option>--filenode=<replaceable>filenode</replaceable></option></term>
+ <listitem>
+ <para>
+ Only validate checksums in the relation with specified relation file node.
+ </para>
Two nits. I would just have been careful about the number of
characters in the line within the <para> markup. And we use
extensively "filenode" in the docs. So the description would become
as follows:
Only validate checksums in the relation with filenode
<replaceable>filenode</replaceable>.
+ [ 'pg_checksums', '--enable', '-filenode', '1234', '-D', $pgdata ],
This fails, but not for the reason it is written for.
It looks strange to not order --filenode alphabetically in --help.
With all these issues cleaned up, I got the attached. Does that look
fine? (I ran pgperltidy and pgindent on top of it.)
--
Michael
Attachment | Content-Type | Size |
---|---|---|
checksums-long-option-3.patch | text/x-diff | 7.8 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Antonin Houska | 2019-05-27 08:43:40 | Re: Converting NOT IN to anti-joins during planning |
Previous Message | Michael Banck | 2019-05-27 08:17:43 | Re: Why does pg_checksums -r not have a long option? |