pgsql: postgres_fdw: Fix corner cases in transaction mode propagation.

From: Etsuro Fujita <efujita(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: postgres_fdw: Fix corner cases in transaction mode propagation.
Date: 2026-10-05 07:17:23
Message-ID: E1xDcwt-00000000NfR-3FLs@gemulon.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

postgres_fdw: Fix corner cases in transaction mode propagation.

This commit addresses the following issues in commit de28140de:

* When the local transaction was DEFERRABLE, begin_remote_xact() started
a remote transaction in the same mode without checking the remote
server version. Since DEFERRABLE is supported only in v9.1 and later,
this would cause a syntax error on older remote servers. Fix this by
propagating the mode only to remote servers that support it.
* pgfdw_xact_callback() committed remote transactions without
synchronizing the read-only mode. If the local transaction had become
READ ONLY since the last remote operation, this would cause deferred
triggers on remote servers to still run in READ WRITE mode. Fix this
by synchronizing the mode at commit time.
* The documentation did not match the actual implementation. Update the
documentation to reflect the implementation.

Reported-by: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
Author: Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com>
Reviewed-by: Matheus Alcantara <matheusssilv97(at)gmail(dot)com>
Discussion: https://postgr.es/m/CAHGQGwE4UMaWGOZq1Z7YUTUSg-CgL4yAzO5YaGwTZe1-6Pjg2g%40mail.gmail.com
Backpatch-through: 19

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/d6393fc40a59830a88a79bdf08e962122fa9c631

Modified Files
--------------
contrib/postgres_fdw/connection.c | 54 +++++++++++++++++++++++---
contrib/postgres_fdw/expected/postgres_fdw.out | 30 ++++++++++++--
contrib/postgres_fdw/sql/postgres_fdw.sql | 30 ++++++++++++--
doc/src/sgml/postgres-fdw.sgml | 15 +++----
4 files changed, 106 insertions(+), 23 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Heikki Linnakangas 2026-10-05 10:18:00 pgsql: Don't sleep with vacuum_delay_point() while holding locks
Previous Message Richard Guo 2026-10-05 06:05:07 pgsql: Fix relids of EC-derived join clauses for lateral UNION ALL memb