Re: Buildfarm failure from overly noisy warning message

Поиск
Список
Период
Сортировка
От Kevin Grittner
Тема Re: Buildfarm failure from overly noisy warning message
Дата
Msg-id 18647993.2978836.1438006952544.JavaMail.yahoo@mail.yahoo.com
обсуждение исходный текст
Ответ на Re: Buildfarm failure from overly noisy warning message  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: Buildfarm failure from overly noisy warning message  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Kevin Grittner <kgrittn@ymail.com> writes:

>> I think a LOG entry when an autovacuum process is actually canceled
>> has value just in case it is happening on a particular table so
>> frequently that the table starts to bloat.  I see no reason to log
>> anything if there is an intention to cancel an autovacuum process
>> but it actually completes before we can do so.

> Hm.  By that logic, I'm not sure if we need anything to be logged here,
> because the autovacuum process will log something about having received
> a query cancel signal.

That seems sufficient to me for normal cases.

> If we're in the business of minimizing log chatter, I'd suggest that
> we remove the entirely-routine "sending cancel" log message, and only
> log something in the uncommon case where the kill() fails (but, per
> original point, reduce that entry to LOG or so; or else print something
> only for not-ESRCH cases).

+1 for only printing for the non-ESRCH cases.

--
Kevin Grittner
EDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



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

Предыдущее
От: Alvaro Herrera
Дата:
Сообщение: Re: Restore-reliability mode
Следующее
От: Tom Lane
Дата:
Сообщение: Re: Failing assertions in indxpath.c, placeholder.c and brin_minmax.c