Re: [HACKERS] Logical replication ApplyContext bloat - Mailing list pgsql-hackers

From Stas Kelvich
Subject Re: [HACKERS] Logical replication ApplyContext bloat
Date
Msg-id 99096066-88A7-434E-B8E3-1D30116DCBD9@postgrespro.ru
Whole thread Raw
In response to Re: [HACKERS] Logical replication ApplyContext bloat  (Peter Eisentraut <peter.eisentraut@2ndquadrant.com>)
List pgsql-hackers
> On 9 May 2017, at 21:53, Peter Eisentraut <peter.eisentraut@2ndquadrant.com> wrote:
>
> On 5/8/17 11:26, Peter Eisentraut wrote:
>> I think it would make more sense to reset ApplyMessageContext in
>> apply_dispatch(), so that it is done consistently after every message
>> (as the name implies), not only some messages.
>
> Committed that.
>
>> Also, perhaps ApplyMessageContext should be a child of
>> TopTransactionContext.  (You have it as a child of ApplyContext, which
>> is under TopMemoryContext.)
>
> Left that as is.

Thanks!

Stas Kelvich
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company





pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [HACKERS] CTE inlining
Next
From: "David G. Johnston"
Date:
Subject: Re: [HACKERS] CTE inlining