Merge pull request #2344 from taosdata/hotfix/rpcAssert
dont reset lockedBy
This commit is contained in:
commit
6d92d8376a
|
@ -574,19 +574,13 @@ static void rpcReleaseConn(SRpcConn *pConn) {
|
||||||
char hashstr[40] = {0};
|
char hashstr[40] = {0};
|
||||||
size_t size = snprintf(hashstr, sizeof(hashstr), "%x:%x:%x:%d", pConn->peerIp, pConn->linkUid, pConn->peerId, pConn->connType);
|
size_t size = snprintf(hashstr, sizeof(hashstr), "%x:%x:%x:%d", pConn->peerIp, pConn->linkUid, pConn->peerId, pConn->connType);
|
||||||
taosHashRemove(pRpc->hash, hashstr, size);
|
taosHashRemove(pRpc->hash, hashstr, size);
|
||||||
|
|
||||||
rpcFreeMsg(pConn->pRspMsg); // it may have a response msg saved, but not request msg
|
rpcFreeMsg(pConn->pRspMsg); // it may have a response msg saved, but not request msg
|
||||||
pConn->pRspMsg = NULL;
|
}
|
||||||
pConn->inType = 0;
|
|
||||||
pConn->inTranId = 0;
|
|
||||||
} else {
|
|
||||||
pConn->outType = 0;
|
|
||||||
pConn->outTranId = 0;
|
|
||||||
pConn->pReqMsg = NULL;
|
|
||||||
}
|
|
||||||
|
|
||||||
taosFreeId(pRpc->idPool, pConn->sid);
|
taosFreeId(pRpc->idPool, pConn->sid);
|
||||||
pConn->pContext = NULL;
|
int64_t lockedBy = pConn->lockedBy;
|
||||||
|
memset(pConn, 0, sizeof(SRpcConn));
|
||||||
|
pConn->lockedBy = lockedBy;
|
||||||
|
|
||||||
tTrace("%s, rpc connection is released", pConn->info);
|
tTrace("%s, rpc connection is released", pConn->info);
|
||||||
}
|
}
|
||||||
|
@ -611,7 +605,6 @@ static SRpcConn *rpcAllocateClientConn(SRpcInfo *pRpc) {
|
||||||
terrno = TSDB_CODE_RPC_MAX_SESSIONS;
|
terrno = TSDB_CODE_RPC_MAX_SESSIONS;
|
||||||
} else {
|
} else {
|
||||||
pConn = pRpc->connList + sid;
|
pConn = pRpc->connList + sid;
|
||||||
memset(pConn, 0, sizeof(SRpcConn));
|
|
||||||
|
|
||||||
pConn->pRpc = pRpc;
|
pConn->pRpc = pRpc;
|
||||||
pConn->sid = sid;
|
pConn->sid = sid;
|
||||||
|
|
Loading…
Reference in New Issue