Ignore rel in MtmGetOldestXmin

knizhnik · knizhnik · commit 317c62194a99 · 2016-03-22T00:28:09.000+03:00
diff --git a/contrib/mmts/multimaster.c b/contrib/mmts/multimaster.c
@@ -312,7 +312,7 @@ Snapshot MtmGetSnapshot(Snapshot snapshot)
 
 TransactionId MtmGetOldestXmin(Relation rel, bool ignoreVacuum)
 {
-    TransactionId xmin = PgGetOldestXmin(rel, ignoreVacuum);
+    TransactionId xmin = PgGetOldestXmin(NULL, ignoreVacuum); /* consider all backends */
     xmin = MtmAdjustOldestXid(xmin);
     return xmin;
 }
@@ -410,7 +410,7 @@ MtmAdjustOldestXid(TransactionId xid)
         
 		MtmLock(LW_EXCLUSIVE);
         ts = (MtmTransState*)hash_search(MtmXid2State, &xid, HASH_FIND, NULL);
-        if (ts != NULL && ts->status == TRANSACTION_STATUS_COMMITTED) { 
+        if (ts != NULL && ts->status == TRANSACTION_STATUS_COMMITTED) {  /* committed transactions have same CSNs at all nodes */
 			csn_t oldestSnapshot;
 			int i;
 

Original file line number	Diff line number	Diff line change
`@@ -312,7 +312,7 @@ Snapshot MtmGetSnapshot(Snapshot snapshot)`
`312`	`312`
`313`	`313`	`TransactionId MtmGetOldestXmin(Relation rel, bool ignoreVacuum)`
`314`	`314`	`{`
`315`		`- TransactionId xmin = PgGetOldestXmin(rel, ignoreVacuum);`
	`315`	`+ TransactionId xmin = PgGetOldestXmin(NULL, ignoreVacuum); /* consider all backends */`
`316`	`316`	`xmin = MtmAdjustOldestXid(xmin);`
`317`	`317`	`return xmin;`
`318`	`318`	`}`
`@@ -410,7 +410,7 @@ MtmAdjustOldestXid(TransactionId xid)`
`410`	`410`
`411`	`411`	`MtmLock(LW_EXCLUSIVE);`
`412`	`412`	`ts = (MtmTransState*)hash_search(MtmXid2State, &xid, HASH_FIND, NULL);`
`413`		`- if (ts != NULL && ts->status == TRANSACTION_STATUS_COMMITTED) {`
	`413`	`+ if (ts != NULL && ts->status == TRANSACTION_STATUS_COMMITTED) { /* committed transactions have same CSNs at all nodes */`
`414`	`414`	`csn_t oldestSnapshot;`
`415`	`415`	`int i;`
`416`	`416`