On Wednesday, December 4, 2024 2:22 PM Michael Paquier <mich...@paquier.xyz> wrote: > > On Wed, Dec 04, 2024 at 11:05:43AM +0530, Amit Kapila wrote: > > On Wed, Dec 4, 2024 at 7:39 AM Zhijie Hou (Fujitsu) > <houzj.f...@fujitsu.com> wrote: > >> It appears there is an additional memory leak caused by allocating > >> publication names within the CacheMemoryContext, as noted in [1]. And > >> it can also be fixed by creating a separate memctx for publication > >> names under the logical decoding context. I think the approach makes > >> sense since the lifespan of publication names should ideally align with > >> that > of the logical decoding context. > > > > Yeah, I don't think we can go with the proposed patch for the local > > memory context as it is. > > Ah, indeed. I was missing your point. Would any of you like to write a patch > to achieve that?
I can try to write a patch if no one else is working on this. Best Regards, Hou zj
