Avoid differences between single thread and multi-thread

Former-commit-id: 848c5ad784d6437533f6c86663b81e41e06a11ea
This commit is contained in:
John Sully 2021-05-26 17:29:01 +00:00
parent b94b46a5ee
commit c091050d4a

View File

@ -834,9 +834,6 @@ int masterTryPartialResynchronization(client *c) {
buflen = snprintf(buf,sizeof(buf),"+CONTINUE\r\n"); buflen = snprintf(buf,sizeof(buf),"+CONTINUE\r\n");
} }
if (connWrite(c->conn,buf,buflen) != buflen) { if (connWrite(c->conn,buf,buflen) != buflen) {
if (FCorrectThread(c))
freeClient(c);
else
freeClientAsync(c); freeClientAsync(c);
return C_OK; return C_OK;
} }
@ -1674,9 +1671,6 @@ void updateSlavesWaitingBgsave(int bgsaveerr, int type)
if (bgsaveerr != C_OK) { if (bgsaveerr != C_OK) {
ul.unlock(); ul.unlock();
if (FCorrectThread(replica))
freeClient(replica);
else
freeClientAsync(replica); freeClientAsync(replica);
serverLog(LL_WARNING,"SYNC failed. BGSAVE child returned an error"); serverLog(LL_WARNING,"SYNC failed. BGSAVE child returned an error");
continue; continue;
@ -1723,9 +1717,6 @@ void updateSlavesWaitingBgsave(int bgsaveerr, int type)
if ((replica->repldbfd = open(g_pserver->rdb_filename,O_RDONLY)) == -1 || if ((replica->repldbfd = open(g_pserver->rdb_filename,O_RDONLY)) == -1 ||
redis_fstat(replica->repldbfd,&buf) == -1) { redis_fstat(replica->repldbfd,&buf) == -1) {
ul.unlock(); ul.unlock();
if (FCorrectThread(replica))
freeClient(replica);
else
freeClientAsync(replica); freeClientAsync(replica);
serverLog(LL_WARNING,"SYNC failed. Can't open/stat DB after BGSAVE: %s", strerror(errno)); serverLog(LL_WARNING,"SYNC failed. Can't open/stat DB after BGSAVE: %s", strerror(errno));
continue; continue;
@ -3677,9 +3668,6 @@ void replicationDiscardCachedMaster(redisMaster *mi) {
serverLog(LL_NOTICE,"Discarding previously cached master state."); serverLog(LL_NOTICE,"Discarding previously cached master state.");
mi->cached_master->flags &= ~CLIENT_MASTER; mi->cached_master->flags &= ~CLIENT_MASTER;
if (FCorrectThread(mi->cached_master))
freeClient(mi->cached_master);
else
freeClientAsync(mi->cached_master); freeClientAsync(mi->cached_master);
mi->cached_master = NULL; mi->cached_master = NULL;
} }