From: | Junwang Zhao <zhjwpku(at)gmail(dot)com> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: fix stale help message |
Date: | 2022-08-14 13:03:15 |
Message-ID: | CAEG8a3LsfZgSzm_yaM8HWVeU-3byaWAyuCVYhLPmU5XSYO49tA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi Michael,
Thanks for your reply :)
I think the goal of `help message` is to tell users(like DBA) how
to use postgres, so it's better to provide a complete view.
I'm not sure by saying `set_plan_disabling_options` do you mean
the postgres source code? If that's the case, I think most of the
installations don't have source code but just binaries, people will
reference the help message by running `postgres --help`.
BTW, I noticed in [0] the doc gives the full options, so I think we
should keep the source code in consistent with the doc.
[0] https://www.postgresql.org/docs/current/app-postgres.html
On Sun, Aug 14, 2022 at 6:17 PM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>
> On Wed, Aug 10, 2022 at 11:32:18PM +0800, Junwang Zhao wrote:
> > when parsing command-line options, the -f option support disabling
> > 8 scan and join methods, o, b and t disable index-only scans,
> > bitmap index scans, and TID scans respectively, add them to the
> > help message.
> >
> > @@ -351,7 +351,7 @@ help(const char *progname)
> > printf(_(" -?, --help show this help, then exit\n"));
> >
> > printf(_("\nDeveloper options:\n"));
> > - printf(_(" -f s|i|n|m|h forbid use of some plan
> > types\n"));
> > + printf(_(" -f s|i|o|b|t|n|m|h forbid use of some plan
> > types\n"));
> > printf(_(" -n do not reinitialize shared
> > memory after abnormal exit\n"));
> > printf(_(" -O allow system table structure
> > changes\n"));
> > printf(_(" -P disable system indexes\n"));
>
> set_plan_disabling_options() is telling that you have all of them, as
> much as the docs. I don't mind fixing that as you suggest, FWIW.
> --
> Michael
--
Regards
Junwang Zhao
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2022-08-14 14:10:41 | Re: latest patches not updated in repos |
Previous Message | Graaff, Selwyn | 2022-08-14 10:29:01 | latest patches not updated in repos |