postgres_fdw: Fix error message for PREPARE TRANSACTION.

Currently, postgres_fdw does not support preparing a remote transaction
for two-phase commit even in the case where the remote transaction is
read-only, but the old error message appeared to imply that that was not
supported only if the remote transaction modified remote tables.  Change
the message so as to include the case where the remote transaction is
read-only.

Also fix a comment above the message.

Also add a note about the lack of supporting PREPARE TRANSACTION to the
postgres_fdw documentation.

Reported-by: Gilles Darold
Author: Gilles Darold and Etsuro Fujita
Reviewed-by: Michael Paquier and Kyotaro Horiguchi
Backpatch-through: 9.4
Discussion: https://postgr.es/m/08600ed3-3084-be70-65ba-279ab19618a5%40darold.net

diff --git a/contrib/postgres_fdw/connection.c b/contrib/postgres_fdw/connection.c
index b4fa89bce906319aaa4bb39f21ba983d5d6ef7d3..0b07a86a6aee5d26673aba3901b141d7ea561376 100644 (file)
--- a/contrib/postgres_fdw/connection.c
+++ b/contrib/postgres_fdw/connection.c
@@ -741,17 +741,17 @@ pgfdw_xact_callback(XactEvent event, void *arg)
                                case XACT_EVENT_PRE_PREPARE:
 
                                        /*
-                                        * We disallow remote transactions that modified anything,
-                                        * since it's not very reasonable to hold them open until
-                                        * the prepared transaction is committed.  For the moment,
-                                        * throw error unconditionally; later we might allow
-                                        * read-only cases.  Note that the error will cause us to
-                                        * come right back here with event == XACT_EVENT_ABORT, so
-                                        * we'll clean up the connection state at that point.
+                                        * We disallow any remote transactions, since it's not
+                                        * very reasonable to hold them open until the prepared
+                                        * transaction is committed.  For the moment, throw error
+                                        * unconditionally; later we might allow read-only cases.
+                                        * Note that the error will cause us to come right back
+                                        * here with event == XACT_EVENT_ABORT, so we'll clean up
+                                        * the connection state at that point.
                                         */
                                        ereport(ERROR,
                                                        (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
-                                                        errmsg("cannot prepare a transaction that modified remote tables")));
+                                                        errmsg("cannot PREPARE a transaction that has operated on postgres_fdw foreign tables")));
                                        break;
                                case XACT_EVENT_PARALLEL_COMMIT:
                                case XACT_EVENT_COMMIT:

diff --git a/doc/src/sgml/postgres-fdw.sgml b/doc/src/sgml/postgres-fdw.sgml
index 7c922821e988ffde8711e8323ff310c8166187d4..48e0ad2756399451315020ff3a2e90dea95b5d9c 100644 (file)
--- a/doc/src/sgml/postgres-fdw.sgml
+++ b/doc/src/sgml/postgres-fdw.sgml
@@ -415,6 +415,12 @@
    COMMITTED</> local transaction.  A future
    <productname>PostgreSQL</productname> release might modify these rules.
   </para>
+
+  <para>
+   Note that it is currently not supported by
+   <filename>postgres_fdw</filename> to prepare the remote transaction for
+   two-phase commit.
+  </para>
  </sect2>
 
  <sect2>