diff --git a/src/inc/tsync.h b/src/inc/tsync.h index d1b68e3f5a1e27b63dd08a4d1d4862c7b1e68179..34c5f29b576e347cf172c3ad5d12111a8296e518 100644 --- a/src/inc/tsync.h +++ b/src/inc/tsync.h @@ -41,7 +41,7 @@ typedef enum { typedef struct { uint32_t nodeId; // node ID assigned by TDengine uint16_t nodePort; // node sync Port - char nodeFqdn[TSDB_FQDN_LEN]; // node FQDN + char nodeFqdn[TSDB_FQDN_LEN * 2]; // node FQDN } SNodeInfo; typedef struct { diff --git a/src/sync/src/syncMain.c b/src/sync/src/syncMain.c index 98cc342af55f4ae2798669ebae02b3844ffba350..35fc11a58389ecd65481a88310fc581cecc7014c 100644 --- a/src/sync/src/syncMain.c +++ b/src/sync/src/syncMain.c @@ -418,8 +418,12 @@ void syncRecover(int64_t rid) { nodeRole = TAOS_SYNC_ROLE_UNSYNCED; (*pNode->notifyRoleFp)(pNode->vgId, nodeRole); - pthread_mutex_lock(&pNode->mutex); - +// pthread_mutex_lock(&pNode->mutex); + int err = pthread_mutex_trylock(&pNode->mutex); + if(EBUSY == err) + { + sDebug("pthread_mutex_trylock failed!"); + } nodeVersion = 0; for (int32_t i = 0; i < pNode->replica; ++i) { @@ -431,7 +435,7 @@ void syncRecover(int64_t rid) { } } - pthread_mutex_unlock(&pNode->mutex); + if(0 == err) pthread_mutex_unlock(&pNode->mutex); syncReleaseNode(pNode); } @@ -987,12 +991,17 @@ static void syncNotStarted(void *param, void *tmrId) { SSyncNode *pNode = pPeer->pSyncNode; - pthread_mutex_lock(&pNode->mutex); +// pthread_mutex_lock(&pNode->mutex); + int err = pthread_mutex_trylock(&pNode->mutex); + if(EBUSY == err) + { + sDebug("pthread_mutex_trylock failed!"); + } pPeer->timer = NULL; pPeer->sstatus = TAOS_SYNC_STATUS_INIT; sInfo("%s, sync conn is still not up, restart and set sstatus:%s", pPeer->id, syncStatus[pPeer->sstatus]); syncRestartConnection(pPeer); - pthread_mutex_unlock(&pNode->mutex); + if(0 == err) pthread_mutex_unlock(&pNode->mutex); syncReleasePeer(pPeer); } @@ -1136,7 +1145,12 @@ static int32_t syncProcessPeerMsg(int64_t rid, void *buffer) { SSyncHead *pHead = buffer; SSyncNode *pNode = pPeer->pSyncNode; - pthread_mutex_lock(&pNode->mutex); +// pthread_mutex_lock(&pNode->mutex); + int err = pthread_mutex_trylock(&pNode->mutex); + if(EBUSY == err) + { + sDebug("pthread_mutex_trylock failed!"); + } int32_t code = syncReadPeerMsg(pPeer, pHead); @@ -1152,7 +1166,7 @@ static int32_t syncProcessPeerMsg(int64_t rid, void *buffer) { } } - pthread_mutex_unlock(&pNode->mutex); + if(0 == err) pthread_mutex_unlock(&pNode->mutex); syncReleasePeer(pPeer); return code; @@ -1245,12 +1259,17 @@ static void syncCheckPeerConnection(void *param, void *tmrId) { SSyncNode *pNode = pPeer->pSyncNode; - pthread_mutex_lock(&pNode->mutex); +// pthread_mutex_lock(&pNode->mutex); + int err = pthread_mutex_trylock(&pNode->mutex); + if(EBUSY == err) + { + sDebug("pthread_mutex_trylock failed!"); + } sDebug("%s, check peer connection", pPeer->id); syncSetupPeerConnection(pPeer); - pthread_mutex_unlock(&pNode->mutex); + if(0 == err) pthread_mutex_unlock(&pNode->mutex); syncReleasePeer(pPeer); } @@ -1331,7 +1350,12 @@ static void syncProcessIncommingConnection(SOCKET connFd, uint32_t sourceIp) { sDebug("vgId:%d, sync connection is incoming, tranId:%u", vgId, msg.tranId); SSyncNode *pNode = *ppNode; - pthread_mutex_lock(&pNode->mutex); +// pthread_mutex_lock(&pNode->mutex); + int err = pthread_mutex_trylock(&pNode->mutex); + if(EBUSY == err) + { + sDebug("pthread_mutex_trylock failed!"); + } SSyncPeer *pPeer; for (i = 0; i < pNode->replica; ++i) { @@ -1362,7 +1386,7 @@ static void syncProcessIncommingConnection(SOCKET connFd, uint32_t sourceIp) { } } - pthread_mutex_unlock(&pNode->mutex); + if(0 == err) pthread_mutex_unlock(&pNode->mutex); } static void syncProcessBrokenLink(int64_t rid, int32_t closedByApp) { @@ -1371,7 +1395,12 @@ static void syncProcessBrokenLink(int64_t rid, int32_t closedByApp) { SSyncNode *pNode = pPeer->pSyncNode; - pthread_mutex_lock(&pNode->mutex); +// pthread_mutex_lock(&pNode->mutex); + int err = pthread_mutex_trylock(&pNode->mutex); + if(EBUSY == err) + { + sDebug("pthread_mutex_trylock failed!"); + } sDebug("%s, TCP link is broken since %s, pfd:%d sfd:%d closedByApp:%d", pPeer->id, strerror(errno), pPeer->peerFd, pPeer->syncFd, closedByApp); @@ -1381,7 +1410,7 @@ static void syncProcessBrokenLink(int64_t rid, int32_t closedByApp) { } syncRestartConnection(pPeer); - pthread_mutex_unlock(&pNode->mutex); + if (0 == err) pthread_mutex_unlock(&pNode->mutex); syncReleasePeer(pPeer); }