Re: not translating backend types for ps and log_line_prefix

From: "Euler Taveira" <euler(at)eulerto(dot)com>
To: Álvaro Herrera <alvherre(at)kurilemu(dot)de>, "Heikki Linnakangas" <hlinnaka(at)iki(dot)fi>
Cc: "Pg Hackers" <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: not translating backend types for ps and log_line_prefix
Date: 2025-12-11 01:27:53
Message-ID: 60fcde5e-2096-4da9-ba73-166c437ef326@app.fastmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Dec 10, 2025, at 6:41 PM, Álvaro Herrera wrote:
> On 2025-Dec-10, Heikki Linnakangas wrote:
>
>> > +PG_PROCTYPE(B_AUTOVAC_LAUNCHER, "autovacuum", gettext_noop("autovacuum launcher"), AutoVacLauncherMain, true)
>> > +PG_PROCTYPE(B_AUTOVAC_WORKER, "autovacuum", gettext_noop("autovacuum worker"), AutoVacWorkerMain, true)
>>
>> It seems not nice that both have the same name, "autovacuum". Similarly, I
>> think it's good to have different names for dead-end backends and regular
>> backends.
>
> True, I was not sure about this either. For context, Euler's patch[2] was
> adding a new column called "category" for use with his new
> representation of log_min_messages, and he decided that something like
> "autovacuum:DEBUG" should affect both the worker and the launcher. I'm
> not sure myself that this is a good idea, because if you want DEBUG for
> just one and not the other, the proposed interface doesn't let you do
> that.
>

It is called "category" to group process types. The main motivation is
"backend" that has multiple entries. For "autovacuum", I checked the code and
launcher has just a few messages so I decided to have just one category. That's
certainly not a strong argument.

> [2] https://postgr.es/m/144f0f96-c6d3-4ad6-a236-00200513e2e0@app.fastmail.com
>
> I'd be okay with instead adding "avlauncher", "avworker", "dead-end",
> "standalone", to differentiate all those types. ("av" sounds perhaps
> too mysterious, so maybe "autovaclauncher" and "autovacworker" for
> those?) Bikeshedding, again, welcome.
>

I'm fine expanding the list. If we carefully choose the names we can certainly
expand it later. For example, the standalone can be a new category in the
future. The same can be applied to autovacuum ("autovacuum" controls the worker
and a new category "autovacumlauncher" controls launcher messages).

--
Euler Taveira
EDB https://www.enterprisedb.com/

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Geoghegan 2025-12-11 01:51:24 Re: Returning nbtree posting list TIDs in DESC order during backwards scans
Previous Message Sami Imseih 2025-12-11 00:57:56 Re: backpatch tests: Rename conflicting role names to 14/15