Remove unneccesary lock

knizhnik · knizhnik · commit 3067e725e222 · 2015-12-28T22:50:06.000+03:00
diff --git a/contrib/multimaster/multimaster.c b/contrib/multimaster/multimaster.c
@@ -211,11 +211,13 @@ static bool TransactionIdIsInDoubt(TransactionId xid)
 		LWLockAcquire(dtm->hashLock, LW_SHARED);
 		inDoubt = hash_search(xid_in_doubt, &xid, HASH_FIND, NULL) != NULL;
 		LWLockRelease(dtm->hashLock);
+#if 0 /* We do not need to wait until transaction locks are released, do we? */
 		if (!inDoubt)
 		{
 			XLogRecPtr lsn;
 			inDoubt = DtmGetTransactionStatus(xid, &lsn) != TRANSACTION_STATUS_IN_PROGRESS;
 		}
+#endif
 		if (inDoubt)
 		{
 			XTM_INFO("Wait for transaction %d to complete\n", xid);

Original file line number	Diff line number	Diff line change
`@@ -211,11 +211,13 @@ static bool TransactionIdIsInDoubt(TransactionId xid)`
`211`	`211`	`LWLockAcquire(dtm->hashLock, LW_SHARED);`
`212`	`212`	`inDoubt = hash_search(xid_in_doubt, &xid, HASH_FIND, NULL) != NULL;`
`213`	`213`	`LWLockRelease(dtm->hashLock);`
	`214`	`+#if 0 /* We do not need to wait until transaction locks are released, do we? */`
`214`	`215`	`if (!inDoubt)`
`215`	`216`	`{`
`216`	`217`	`XLogRecPtr lsn;`
`217`	`218`	`inDoubt = DtmGetTransactionStatus(xid, &lsn) != TRANSACTION_STATUS_IN_PROGRESS;`
`218`	`219`	`}`
	`220`	`+#endif`
`219`	`221`	`if (inDoubt)`
`220`	`222`	`{`
`221`	`223`	`XTM_INFO("Wait for transaction %d to complete\n", xid);`