Re: Patch to address creation of PgStat* contexts with null parent context - Mailing list pgsql-hackers

From Andres Freund
Subject Re: Patch to address creation of PgStat* contexts with null parent context
Date
Msg-id 20220917161006.v5b5obop2ojgqiv2@awork3.anarazel.de
Whole thread Raw
In response to Re: Patch to address creation of PgStat* contexts with null parent context  ("Drouvot, Bertrand" <bdrouvot@amazon.com>)
Responses Re: Patch to address creation of PgStat* contexts with null parent context
List pgsql-hackers
Hi,

On 2022-09-07 11:11:11 +0200, Drouvot, Bertrand wrote:
> On 9/6/22 7:53 AM, Kyotaro Horiguchi wrote:
> > At Mon, 5 Sep 2022 14:46:55 +0200, "Drouvot, Bertrand"<bdrouvot@amazon.com>  wrote in
> > > Looks like that both approaches have their pros and cons. I'm tempted
> > > to vote +1 on "just changing" the parent context to TopMemoryContext
> > > and not changing the allocations locations.
> > Yeah. It is safe more than anything and we don't have a problem there.
> > 
> > So, I'm fine with just replacing the parent context at the three places.
> 
> Attached a patch proposal to do so.

Pushed. Thanks for the report and the fix!

Greetings,

Andres Freund



pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: archive modules
Next
From: Michael Banck
Date:
Subject: Re: [EXTERNAL] Re: Support load balancing in libpq