On Tue, Dec 10, 2024 at 03:24:19AM +0000, vignesh C wrote: > Yes, that makes sense. How about something like the attached patch.
So you have this bit hidden in 7f481b8d3884, causing a small conflict
when cherry-picking the change from 15 to 14:
- oldctx = MemoryContextSwitchTo(CacheMemoryContext);
- if (data->publications)
- list_free_deep(data->publications);
+ static MemoryContext pubctx = NULL;
[...]
- if (data->publications)
- {
- list_free_deep(data->publications);
- data->publications = NIL;
- }
+ static MemoryContext pubctx = NULL;
Your versions look OK at quick glance, for the publication list
caching.
--
Michael
signature.asc
Description: PGP signature
