Disallow committing a prepared transaction unless we are in the same database

it was executed in.  Someday it might be nice to allow cross-DB commits, but
work would be needed in NOTIFY and perhaps other places.  Per Heikki.

diff --git a/src/backend/access/transam/twophase.c b/src/backend/access/transam/twophase.c
index 45220d119642c033e9c56f8ed017577ba1ffd66f..55f6b88bcd0799ccaa7c32a975536c0bc2e1005c 100644 (file)
--- a/src/backend/access/transam/twophase.c
+++ b/src/backend/access/transam/twophase.c
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  * IDENTIFICATION
- *             $PostgreSQL: pgsql/src/backend/access/transam/twophase.c,v 1.27 2007/01/16 13:28:56 alvherre Exp $
+ *             $PostgreSQL: pgsql/src/backend/access/transam/twophase.c,v 1.28 2007/02/13 19:39:42 tgl Exp $
  *
  * NOTES
  *             Each global transaction is associated with a global transaction
@@ -393,6 +393,18 @@ LockGXact(const char *gid, Oid user)
                                  errmsg("permission denied to finish prepared transaction"),
                                         errhint("Must be superuser or the user that prepared the transaction.")));
 
+               /*
+                * Note: it probably would be possible to allow committing from another
+                * database; but at the moment NOTIFY is known not to work and there
+                * may be some other issues as well.  Hence disallow until someone
+                * gets motivated to make it work.
+                */
+               if (MyDatabaseId != gxact->proc.databaseId)
+                       ereport(ERROR,
+                                       (errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
+                                        errmsg("prepared transaction belongs to another database"),
+                                        errhint("Connect to the database where the transaction was prepared to finish it.")));
+
                /* OK for me to lock it */
                gxact->locking_xid = GetTopTransactionId();