Re: convert elog(LOG) calls to ereport

Поиск
Список
Период
Сортировка
От Peter Eisentraut
Тема Re: convert elog(LOG) calls to ereport
Дата
Msg-id c1a3656b-a596-05ec-2353-3dfc3657eae9@enterprisedb.com
обсуждение исходный текст
Ответ на Re: convert elog(LOG) calls to ereport  (Michael Paquier <michael@paquier.xyz>)
Список pgsql-hackers
On 05.12.20 03:22, Michael Paquier wrote:
> On Fri, Dec 04, 2020 at 02:34:26PM +0100, Peter Eisentraut wrote:
>> On 2020-12-02 15:04, Alvaro Herrera wrote:
>>> I do wonder if it'd be a good idea to move the syscall
>>> name itself out of the message, too; that would reduce the number of
>>> messages to translate 50x to just "%s(%s) failed: %m" instead of one
>>> message per distinct syscall.
>>
>> Seems useful, but perhaps as a separate project.
> 
> -       elog(LOG, "getsockname() failed: %m");
> +       ereport(LOG,
> +               (errmsg("getsockname() failed: %m")));
> FWIW, I disagree with the approach taken by eb93f3a.  As of HEAD, it
> is now required to translate all those strings.  I think that it would
> have been better to remove the function names from all those error
> messages and not require the same pattern to be translated N times.

I made another pass across this and implemented the requested change.



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

Предыдущее
От: Amit Langote
Дата:
Сообщение: Re: Table refer leak in logical replication
Следующее
От: Tom Lane
Дата:
Сообщение: Re: [bug?] Missed parallel safety checks, and wrong parallel safety