Re: isolationtester: add session name to application name

From: Andres Freund <andres(at)anarazel(dot)de>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: isolationtester: add session name to application name
Date: 2021-12-13 18:48:34
Message-ID: 20211213184834.acltkdc2h4yj5npc@alap3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On 2021-12-13 19:46:34 +0900, Michael Paquier wrote:
> On Fri, Dec 10, 2021 at 05:20:52PM -0800, Andres Freund wrote:
> > These days isolationtester.c already prefixes log output with the session
> > name. How about doing the same for application_name? It's a *tad* more
> > complicated than I'd like because isolationtester.c currently doesn't know the
> > name of the test its executing.
>
> +1 for the idea. Maybe it could be backpatched?

Not entirely trivially - the changes have some dependencies on other changes
(e.g. b1907d688, more on 741d7f104, but that was backpatched). I guess we
could backpatch b1907d688 as well, but I'm not sure its worth it?

> > + * easier to map spec file sesions to log output and
>
> One s/sesions/sessions/ here.

Ah, good catch.

Greetings,

Andres Freund

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2021-12-13 18:52:57 Re: Add client connection check during the execution of the query
Previous Message Bossart, Nathan 2021-12-13 18:30:43 Re: O(n) tasks cause lengthy startups and checkpoints