You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@ratis.apache.org by "OneSizeFitsQuorum (via GitHub)" <gi...@apache.org> on 2024/09/05 03:06:18 UTC

Re: [PR] Demo md5 mismatch when snapshot install [ratis]

OneSizeFitsQuorum commented on PR #1142:
URL: https://github.com/apache/ratis/pull/1142#issuecomment-2330507158

   We spent a lot of time troubleshooting this issue, and what we were able to confirm in the end is that there seems to be some problem with `digester.reset`, which causes the final MD5 calculation to be incorrect. We searched for related issues on the OpenJDK official [website](https://bugs.openjdk.org/browse/JDK-8300416?jql=project%20%3D%20JDK%20AND%20issuetype%20%3D%20Bug%20AND%20affectedVersion%20in%20releasedVersions()%20AND%20text%20~%20%22MD5%22%20ORDER%20BY%20created%20DESC) but couldn't find an exact cause.
   
   However, we found that instead of using reset, creating a new digester each time a new file appears should solve the issue (our testing process can verify this). Therefore, I recommend making this change directly. 
   
   What's your opinion? @szetszwo 


-- 
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-unsubscribe@ratis.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org