Fix 2pc commit of transaction updating temporary table

knizhnik · kelvich · commit 8d514e8886d7 · 2017-11-13T02:17:35.000+03:00
diff --git a/multimaster.c b/multimaster.c
@@ -1133,7 +1133,7 @@ MtmPostPrepareTransaction(MtmCurrentTrans* x)
 		MTM_TXTRACE(x, "recovery? 5");
 		MtmResetTransaction();
 		MTM_TXTRACE(x, "recovery? 6");
-	} else { 
+	} else if (!ts->isLocal)  { 
 		MTM_TXTRACE(x, "not recovery?");
 		Mtm2PCVoting(x, ts);
 		MtmUnlock();
@@ -1170,18 +1170,19 @@ MtmPreCommitPreparedTransaction(MtmCurrentTrans* x)
 		ts = tm->state;
 
 		Assert(MtmIsCoordinator(ts));
+		if (!ts->isLocal) { 
+			ts->votingCompleted = false;
+			ts->votedMask = 0;
+			ts->procno = MyProc->pgprocno;
+			MTM_LOG2("Coordinator of transaction %s sends MSG_PRECOMMIT", ts->gid);
+			Assert(replorigin_session_origin == InvalidRepOriginId);
+			MtmUnlock();
+			SetPreparedTransactionState(ts->gid, MULTIMASTER_PRECOMMITTED);
+			//MtmSend2PCMessage(ts, MSG_PRECOMMIT);
+			MtmLock(LW_EXCLUSIVE);
 
-		ts->votingCompleted = false;
-		ts->votedMask = 0;
-		ts->procno = MyProc->pgprocno;
-		MTM_LOG2("Coordinator of transaction %s sends MSG_PRECOMMIT", ts->gid);
-		Assert(replorigin_session_origin == InvalidRepOriginId);
-		MtmUnlock();
-		SetPreparedTransactionState(ts->gid, MULTIMASTER_PRECOMMITTED);
-		//MtmSend2PCMessage(ts, MSG_PRECOMMIT);
-		MtmLock(LW_EXCLUSIVE);
-
-		Mtm2PCVoting(x, ts);
+			Mtm2PCVoting(x, ts);
+		}
 
 		x->xid = ts->xid;
 		x->csn = ts->csn;
diff --git a/tests2/test_regression.py b/tests2/test_regression.py
@@ -15,7 +15,7 @@ def setUpClass(self):
     @classmethod
     def tearDownClass(self):
         print('tearDown')
-        subprocess.check_call(['docker-compose','down'])
+#        subprocess.check_call(['docker-compose','down'])
 
     def test_regression(self):
         # XXX: make smth clever here
