F64116045 commented on PR #11199: URL: https://github.com/apache/ozone/pull/11199#issuecomment-5534120575
> Its simple to adjust the position in the processing chain after the Check Handler is written so we can always change that small part. That makes sense, thanks. So I’m planning to place the handler here for now: ```java .addNext(ecReplicationCheckHandler) .addNext(checksumMismatchHandler) .addNext(ratisReplicationCheckHandler) ``` The earlier handlers won’t stop the CLOSED RATIS containers this check applies to, and the checksum handler will return false so the rest of the chain can continue. Please let me know if you have any other concerns, thanks! (BTW, since this refactor will replace much of the current implementation, I plan to amend the commit and force-push the updated patch after) -- 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]
