aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2001-05-04 18:39:16 +0000
committerTom Lane <tgl@sss.pgh.pa.us>2001-05-04 18:39:16 +0000
commite2e19ca0cdb47a7359f29359b8d47f0a298b22f6 (patch)
tree6df411d6f44311f9d5242108b459c88173ca7662
parent602b742507f9598b1d3ba1c8d9a016ac101fafbc (diff)
downloadpostgresql-e2e19ca0cdb47a7359f29359b8d47f0a298b22f6.tar.gz
postgresql-e2e19ca0cdb47a7359f29359b8d47f0a298b22f6.zip
Seems like we should not hold off cancel/die interrupts while we are
running deferred triggers. They are really part of the regular transaction, and they could take awhile.
-rw-r--r--src/backend/access/transam/xact.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/src/backend/access/transam/xact.c b/src/backend/access/transam/xact.c
index c88e665a7ec..4e13d689f1f 100644
--- a/src/backend/access/transam/xact.c
+++ b/src/backend/access/transam/xact.c
@@ -8,7 +8,7 @@
*
*
* IDENTIFICATION
- * $Header: /cvsroot/pgsql/src/backend/access/transam/xact.c,v 1.101 2001/03/22 06:16:10 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/access/transam/xact.c,v 1.102 2001/05/04 18:39:16 tgl Exp $
*
* NOTES
* Transaction aborts can now occur two ways:
@@ -1026,9 +1026,6 @@ CommitTransaction(void)
if (s->state != TRANS_INPROGRESS)
elog(NOTICE, "CommitTransaction and not in in-progress state ");
- /* Prevent cancel/die interrupt while cleaning up */
- HOLD_INTERRUPTS();
-
/*
* Tell the trigger manager that this transaction is about to be
* committed. He'll invoke all trigger deferred until XACT before we
@@ -1036,6 +1033,9 @@ CommitTransaction(void)
*/
DeferredTriggerEndXact();
+ /* Prevent cancel/die interrupt while cleaning up */
+ HOLD_INTERRUPTS();
+
/*
* set the current transaction state information appropriately during
* the abort processing