Avoid call raftable with infinite timeout

knizhnik · kelvich · commit 0e9e9ca700f9 · 2017-11-13T00:17:08.000+03:00
diff --git a/multimaster.c b/multimaster.c
@@ -1347,7 +1347,7 @@ bool MtmRefreshClusterStatus(bool nowait)
 	int clique_size;
 	int i;
 
-	if (!MtmUseRaftable || !MtmBuildConnectivityMatrix(matrix, nowait)) { 
+	if (!MtmBuildConnectivityMatrix(matrix, nowait)) { 
 		/* RAFT is not available */
 		return false;
 	}
@@ -1436,11 +1436,12 @@ void MtmOnNodeDisconnect(int nodeId)
 	BIT_SET(Mtm->reconnectMask, nodeId-1);
 	MtmUnlock();
 
-	RaftableSet(psprintf("node-mask-%d", MtmNodeId), &Mtm->connectivityMask, sizeof Mtm->connectivityMask, false);
+	RaftableSet(psprintf("node-mask-%d", MtmNodeId), &Mtm->connectivityMask, sizeof Mtm->connectivityMask, true); /* false); -- TODO: raftable is hanged with nowait=true */
 
 	MtmSleep(MSEC_TO_USEC(MtmRaftPollDelay));
 
-	if (!MtmRefreshClusterStatus(false)) { 
+	if (!MtmUseRaftable) 
+	{
 		MtmLock(LW_EXCLUSIVE);
 		if (!BIT_CHECK(Mtm->disabledNodeMask, nodeId-1)) { 
 			MtmDisableNode(nodeId);
@@ -1457,7 +1458,9 @@ void MtmOnNodeDisconnect(int nodeId)
 			}
 		}
 		MtmUnlock();
-	}
+	} else { 
+		MtmRefreshClusterStatus(true); /* false); -- TODO: raftable can handg in nowait=true */
+    }
 }
 
 void MtmOnNodeConnect(int nodeId)
@@ -1467,7 +1470,7 @@ void MtmOnNodeConnect(int nodeId)
 	MtmUnlock();
 
 	MTM_LOG1("Reconnect node %d", nodeId);
-	RaftableSet(psprintf("node-mask-%d", MtmNodeId), &Mtm->connectivityMask, sizeof Mtm->connectivityMask, false);
+	RaftableSet(psprintf("node-mask-%d", MtmNodeId), &Mtm->connectivityMask, sizeof Mtm->connectivityMask, true); /* false); -- TODO: raftable is hanged with nowait=true */
 }
 
 
