Fix wrong assertion in MtmEndTransaction().

kelvich · kelvich · commit cef84ed39e68 · 2017-10-15T02:38:01.000+03:00
MtmBroadcastPollMessage() also happens when old prepared tx are
restored, but we are still in DISABLED mode. So it is legal
to finish in-progress transaction.
diff --git a/contrib/mmts/multimaster.c b/contrib/mmts/multimaster.c
@@ -1530,7 +1530,7 @@ MtmEndTransaction(MtmCurrentTrans* x, bool commit)
 						 (commit ? "commit" : "rollback"), ts->gid, ts->xid, ts->gtid.node, ts->gtid.xid, MtmTxnStatusMnem[ts->status]);
 			if (commit) {
 				if (!(ts->status == TRANSACTION_STATUS_UNKNOWN
-					  || (ts->status == TRANSACTION_STATUS_IN_PROGRESS && Mtm->status == MTM_RECOVERY)))
+					  || (ts->status == TRANSACTION_STATUS_IN_PROGRESS && Mtm->status <= MTM_RECOVERY)))
 				{
 					MtmUnlock();
 					MTM_ELOG(ERROR, "Attempt to commit %s transaction %s (%llu)",

Original file line number	Diff line number	Diff line change
`@@ -1530,7 +1530,7 @@ MtmEndTransaction(MtmCurrentTrans* x, bool commit)`
`1530`	`1530`	`(commit ? "commit" : "rollback"), ts->gid, ts->xid, ts->gtid.node, ts->gtid.xid, MtmTxnStatusMnem[ts->status]);`
`1531`	`1531`	`if (commit) {`
`1532`	`1532`	`if (!(ts->status == TRANSACTION_STATUS_UNKNOWN`
`1533`		`- \|\| (ts->status == TRANSACTION_STATUS_IN_PROGRESS && Mtm->status == MTM_RECOVERY)))`
	`1533`	`+ \|\| (ts->status == TRANSACTION_STATUS_IN_PROGRESS && Mtm->status <= MTM_RECOVERY)))`
`1534`	`1534`	`{`
`1535`	`1535`	`MtmUnlock();`
`1536`	`1536`	`MTM_ELOG(ERROR, "Attempt to commit %s transaction %s (%llu)",`