Re: Log prefix missing for subscriber log messages received from publisher

On 2025/07/18 1:15, Álvaro Herrera wrote:
> On 2025-Jul-18, Fujii Masao wrote:
> 
>> The current patches add nearly identical notice_receiver functions
>> in multiple places such as libpqwalreceiver.c and elsewhere. To avoid
>> duplicating the same logic, could we define a shared notice receiver
>> function in a common file, like libpq-be-fe-helpers.h, and use it in
>> all three locations?
> 
> I like the idea of reducing duplication.  I don't like the "prefix"
> as proposed though, because it's not going to work well for translation
> (string building) -- I'd rather pass the entire phrase from caller, so
> that the translator has one piece to translate which lives in the module
> that emits it.  I think I'd do something like
> 
>      ereport(LOG,
>              errmsg_internal("%s: %.*s",
> 	                    _(prefix), len, message));
> 
> and see to it that each caller uses gettext_noop() around the string
> they pass as "arg", for gettext to collect.

Agreed. Thanks!

Regards,

-- 
Fujii Masao
NTT DATA Japan Corporation