On Sat, 2026-03-21 at 16:25 +0530, Amit Kapila wrote:
> > > Maybe another possibility would be to use a separate memory
> > > context
> > > for each subscription, initially making it a child of the
> > > transaction
> > > context, and then reparenting it as appropriate.
> >
> > I mean something like this on top of your 0003.
> >
>
> +1. This approach and patch looks like a better way to deal with this
> issue.
Thank you, pushed.
A couple minor adjustments: in GetSubscription(), I create the context
after the early return, in case the subscription isn't found. Also, I
combined the:
if (newsub)
...
if (!newsub)
...
into if/else.
The only remaining issue in this thread is how to make
test_fdw_connect() available during the pg_dump tests without polluting
pg_proc. Is there a reasonable way to do that?
Regards,
Jeff Davis
