Re: pg_autovacuum integration patch

Поиск
Список
Период
Сортировка
От Andrew Dunstan
Тема Re: pg_autovacuum integration patch
Дата
Msg-id 40D05767.3060802@dunslane.net
обсуждение исходный текст
Ответ на Re: pg_autovacuum integration patch  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-patches
Tom Lane wrote:

>"Magnus Hagander" <mha@sollentuna.net> writes:
>
>
>>I also noticed:
>>!         elog(ERROR, "pg_autovacuum: GUC variable stats_row_level
>>must be enabled.");
>>!         elog(ERROR, "       Please fix the problems and try
>>again.");
>>
>>
>
>
>
>>If you use the ereport() call instead of elog, you can set the second
>>one as a HINT. Since it's really the same error, I don't think you want
>>multiple errors logged...
>>
>>
>
>Even more to the point, control will never *reach* the second elog().
>Matthew clearly needs to spend more time studying the backend error
>message reporting mechanism.  There is some documentation here:
>http://developer.postgresql.org/docs/postgres/error-message-reporting.html
>and the backend code is certainly chock-full of examples.
>
>
>

Perhaps it's just as well that it isn't reached :-) "Please fix the
problems and try again" doesn't strike me as a very useful message.

cheers

andrew

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

Предыдущее
От: "Matthew T. O'Connor"
Дата:
Сообщение: Re: pg_autovacuum integration patch
Следующее
От: "Dave Page"
Дата:
Сообщение: Re: pg_ctl service integration for WIN32