Re: Patch to address creation of PgStat* contexts with null parent context

Поиск
Список
Период
Сортировка
От Andres Freund
Тема Re: Patch to address creation of PgStat* contexts with null parent context
Дата
Msg-id 20220917161006.v5b5obop2ojgqiv2@awork3.anarazel.de
обсуждение исходный текст
Ответ на Re: Patch to address creation of PgStat* contexts with null parent context  ("Drouvot, Bertrand" <bdrouvot@amazon.com>)
Ответы Re: Patch to address creation of PgStat* contexts with null parent context  ("Drouvot, Bertrand" <bertranddrouvot.pg@gmail.com>)
Список 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 по дате отправления:

Предыдущее
От: Peter Eisentraut
Дата:
Сообщение: Re: archive modules
Следующее
От: Michael Banck
Дата:
Сообщение: Re: [EXTERNAL] Re: Support load balancing in libpq