On Sun, Mar 15, 2026 at 4:25 AM Jeff Davis <[email protected]> wrote: > > Attached series including patches to address Andres's and Amit's > comments, too. >
0001 LGTM. 0003: @@ -5056,8 +5058,15 @@ maybe_reread_subscription(void) started_tx = true; } - /* Ensure allocations in permanent context. */ - oldctx = MemoryContextSwitchTo(ApplyContext); + newctx = AllocSetContextCreate(ApplyContext, + "Subscription Context", + ALLOCSET_SMALL_SIZES); + Won't it be sufficient if we just reset MySubscriptionCtx here or in callback subscription_change_cb()? -- With Regards, Amit Kapila.
