On Thursday, December 5, 2024 12:52 PM Michael Paquier <mich...@paquier.xyz> wrote:
Hi, > > On Thu, Dec 05, 2024 at 04:31:56AM +0000, Zhijie Hou (Fujitsu) wrote: > > I realized that this patch cannot be backpatched because it introduces > > a new field into the public PGOutputData structure. Therefore, I think > > we may need to use Alvaro's version [1] for the back branches. > > > > Thanks for the patch. > > For HEAD it should be as good as it can be as it avoids the problem of > CacheMemoryContext bloating for your case and my case. Alvaro's patch > would not take care of your case, unfortunately, but I'm less worried about > this > case in the back branches and we don't track the parent context where > StartupDecodingContext() has begun its work when building PGOutputData. > Thoughts? I am fine with the plan. Thanks. Best Regards, Hou zj
