Re: Emit a warning if the extension's GUC is set incorrectly

Поиск
Список
Период
Сортировка
От Daniel Gustafsson
Тема Re: Emit a warning if the extension's GUC is set incorrectly
Дата
Msg-id 681A95E0-0CCA-4C66-A12B-CE0D8D5076D7@yesql.se
обсуждение исходный текст
Ответ на Emit a warning if the extension's GUC is set incorrectly  (Shinya Kato <Shinya11.Kato@oss.nttdata.com>)
Ответы Re: Emit a warning if the extension's GUC is set incorrectly  (Shinya Kato <Shinya11.Kato@oss.nttdata.com>)
Список pgsql-hackers
> On 14 Nov 2021, at 11:03, Shinya Kato <Shinya11.Kato@oss.nttdata.com> wrote:

> If wrong GUCs of auth_delay, pg_trgm, postgres_fdw and sepgsql are described in postgresql.conf, a warning is not
emittedunlike pg_stat_statements, auto_explain and pg_prewarm. 
> So, I improved it by adding EmitWarningsOnPlaceholders.
> An example output is shown below.
> ---
> 2021-11-14 18:18:16.486 JST [487067] WARNING:  unrecognized configuration parameter "auth_delay.xxx"
> ---
>
> What do you think?

Seems reasonable on a quick skim, commit da2c1b8a2 did a similar roundup back
in 2009 but at the time most of these didn't exist (pg_trgm did but didn't have
custom option back then).  There is one additional callsite defining custom
variables in src/pl/tcl/pltcl.c which probably should get this treatment as
well, it would align it with the pl/perl counterpart.

I'll have a closer look and test tomorrow.

--
Daniel Gustafsson        https://vmware.com/




В списке pgsql-hackers по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: Commitfest 2021-11 Patch Triage - Part 2
Следующее
От: Victor Spirin
Дата:
Сообщение: Re: Atomic rename feature for Windows.