greatsharp commented on code in PR #3077:
URL: https://github.com/apache/kvrocks/pull/3077#discussion_r2366003014


##########
src/cluster/batch_sender.cc:
##########
@@ -100,7 +100,7 @@ Status BatchSender::sendApplyBatchCmd(int fd, const 
rocksdb::WriteBatch &write_b
 
   GET_OR_RET(util::SockSend(fd, redis::ArrayOfBulkStrings({"APPLYBATCH", 
write_batch.Data()})));
 
-  std::string line = GET_OR_RET(util::SockReadLine(fd));
+  std::string line = GET_OR_RET(util::SockReadLineWithRetry(fd, 10, 500));

Review Comment:
   > My case is a bit different, I get this error in `checkMultipleResponses` 
and I think that may be due to the 1s timeout setting.
   > 
   > What is the full error message you see? I think it can come from several 
places.. Also I am not sure if evbuffer should be used on a blocking 
connection, that lib is intended for non-blocking fd.
   
   please use raw-key-value migrate type



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to