Re: Log prefix missing for subscriber log messages received from publisher
Fujii Masao <masao.fujii@gmail.com>
From: Fujii Masao <masao.fujii@gmail.com>
To: Álvaro Herrera <alvherre@kurilemu.de>
Cc: Fujii Masao <masao.fujii@oss.nttdata.com>,
vignesh C <vignesh21@gmail.com>, pgsql-hackers <pgsql-hackers@lists.postgresql.org>
Date: 2025-07-22T12:21:10Z
Lists: pgsql-hackers
Attachments
- v1-0001-Remove-translation-marker-from-libpq-be-fe-helper.patch (application/octet-stream) patch v1-0001
On Tue, Jul 22, 2025 at 8:47 PM Álvaro Herrera <alvherre@kurilemu.de> wrote:
>
> On 2025-Jul-22, Fujii Masao wrote:
>
> > Is the issue caused by the use of _() in libpq-be-fe-helpers.h,
> > instead of using gettext_noop() in dblink.c? At least in my test
> > environment, the build error disappeared after applying
> > the following change:
> >
> > - errmsg_internal("%s: %.*s", _(prefix), len, message));
> > + errmsg_internal("%s: %.*s", prefix, len, message));
>
> Oh yeah, I should have remembered this -- see commit 213c959a294d. Feel
> free to do away with the whole translation thing ... doesn't seem worth
> spending more time on it.
Yes! The attached patch removes the translation marker and the
gettext_noop() calls I recently added.
> We should still remove all gettext_noop() markers in contrib :-)
You mean to remove gettext_noop() also from basic_archive.c?
Regards,
--
Fujii Masao
Commits
Same data as JSON:
GET /api/v1/messages/:b64id/commits
the thread's linked commits as JSON, with link sources.
API reference →
-
Remove translation marker from libpq-be-fe-helpers.h.
- a7ca73af662b 19 (unreleased) landed
-
Log remote NOTICE, WARNING, and similar messages using ereport().
- 112faf1378ee 19 (unreleased) landed
-
Remove translation markers from libpq-be-fe-helpers.h
- 213c959a294d 17.0 cited