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@paquier.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
