mikemccand commented on issue #16674: URL: https://github.com/apache/lucene/issues/16674#issuecomment-6018767712
I think there is another full read from `checkIntegrity`? A step 0, run (sequentially?) on all the input segments before they are merged? That is a scan of the whole file and then confirming the checksum? But it's nearly redundant because any of the existing steps 1, 2, 3 could validate the checksum after their full scan? But ... it'd be a change in error detection timing because it would have done some merge work before detecting the bit flips. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
