133tosakarin commented on code in PR #1142: URL: https://github.com/apache/ratis/pull/1142#discussion_r1743430717
########## ratis-server/src/main/java/org/apache/ratis/server/storage/SnapshotManager.java: ########## @@ -114,7 +115,8 @@ public void installSnapshot(InstallSnapshotRequestProto request, StateMachine st // TODO: Make sure that subsequent requests for the same installSnapshot are coming in order, // and are not lost when whole request cycle is done. Check requestId and requestIndex here - + MessageDigest digester = newMd5Digest(); + digester.reset(); Review Comment: well! I am testing in our environment to see if there are still problems. Before that, I will convert this PR to draft -- 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: issues-unsubscr...@ratis.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org