Re: Fix GUC_NO_SHOW_ALL test scenario in 003_check_guc.pl

Justin Pryzby <pryzby(at)telsasoft(dot)com> writes:
> On Thu, Feb 09, 2023 at 10:28:14AM +0900, Michael Paquier wrote:
>> I am wondering.. Did people notice that this adds GUC_NOT_IN_SAMPLE
>> to config_file in guc_tables.c? This makes sense in the long run
>> based on what this parameter is by design, still there may be an
>> objection to doing that?

> I think it's fine to add the flag.

Hm. On the one hand, if it is in fact not in postgresql.conf.sample,
then that flag should be set for sure. OTOH I see that that flag
isn't purely documentation: help_config.c thinks it should hide
GUCs that are marked that way. Do we really want that behavior?
Not sure. I can see an argument that you might want --describe-config
to tell you that, but there are a lot of other GUC_NOT_IN_SAMPLE
GUCs that maybe do indeed deserve to be left out.

regards, tom lane