Re: EmitWarningsOnPlaceholders is too quiet

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: EmitWarningsOnPlaceholders is too quiet
Дата
Msg-id 26710.1231191064@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: EmitWarningsOnPlaceholders is too quiet  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
I wrote:
> I rememebered what INFO is for: it's the elevel that VACUUM VERBOSE
> uses to ensure that its output gets seen at the client.  (I missed
> that in my first grep because vacuum's elog/ereport calls don't use
> INFO as a hard-wired elevel.)  So we probably can't get rid of it.
> But that makes it even more clear IMHO that EmitWarningsOnPlaceholders
> shouldn't be using it.  It's not intended for error/warning types of
> uses, but for verbose logging.

After a more careful grep, here are all the uses of INFO elog level:

VACUUM/ANALYZE/CLUSTER VERBOSE
NOTIFY, in a standalone backend
guc.c's set_config_option, to bleat about bad config settings obtained fromALTER USER or ALTER DATABASE during
connectionstartup
 
guc.c's EmitWarningsOnPlaceholders, to bleat about bad custom variables

Also, most of the PLs have ways for users to emit messages at INFO
elevel, but whether a user uses that in a consistent way is not our
problem.

I'm of the opinion that guc.c is in error here, and what it should be
using for its purposes is WARNING.  Objections?
        regards, tom lane


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

Предыдущее
От: "Joshua D. Drake"
Дата:
Сообщение: Re: QuickLZ compression algorithm (Re: Inclusion in the PostgreSQL backend for toasting rows)
Следующее
От: Martin Pihlak
Дата:
Сообщение: Re: SQL/MED compatible connection manager