Re: pg_autovacuum integration patch - Mailing list pgsql-patches

From Andrew Dunstan
Subject Re: pg_autovacuum integration patch
Date
Msg-id 40D05767.3060802@dunslane.net
Whole thread Raw
In response to Re: pg_autovacuum integration patch  (Tom Lane <tgl@sss.pgh.pa.us>)
List 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 by date:

Previous
From: "Matthew T. O'Connor"
Date:
Subject: Re: pg_autovacuum integration patch
Next
From: "Dave Page"
Date:
Subject: Re: pg_ctl service integration for WIN32