| From: | Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> |
|---|---|
| To: | Nikolay Samokhvalov <nik(at)postgres(dot)ai> |
| Cc: | Matheus Alcantara <matheusssilv97(at)gmail(dot)com>, Fujii Masao <masao(dot)fujii(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: postgres_fdw: transaction mode inheritance corner cases |
| Date: | 2026-10-01 17:32:44 |
| Message-ID: | CAPmGK14s4dS5FYr96vkK_Ti-sVFi8c0V5_kmDPv7mEQbEXEMvg@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Fri, Oct 2, 2026 at 1:47 AM Nikolay Samokhvalov <nik(at)postgres(dot)ai> wrote:
> On Wed, Sep 30, 2026 at 11:27 AM Matheus Alcantara
> <matheusssilv97(at)gmail(dot)com> wrote:
> > With the patch, the new begin_remote_xact() call in create_cursor opens a
> > remote SAVEPOINT s2 and creates the cursor inside it. ROLLBACK TO s1 then
> > destroys the remote cursor while the local side still thinks it exists.
>
> Thanks, Matheus. My AI harness for testing reproduced this on
> REL_19_STABLE at 9e73b209 with Etsuro's v1 patch and prepared the
> attached incremental patch. It declares the remote cursor before
> advancing the remote savepoint level, then synchronizes the transaction
> mode before FETCH. The second FETCH fails with 34000 on v1 and succeeds
> with this patch.
Will look into the patch. Thanks!
Best regards,
Etsuro Fujita
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Etsuro Fujita | 2026-10-01 17:37:29 | Re: postgres_fdw: transaction mode inheritance corner cases |
| Previous Message | Etsuro Fujita | 2026-10-01 17:30:33 | Re: postgres_fdw: transaction mode inheritance corner cases |