sodonnel commented on PR #11199: URL: https://github.com/apache/ozone/pull/11199#issuecomment-5732127443
The other PR has been merged removing the transient state from ReplicationManagerReport, so this PR can be updated based on current master. If we are going to report BCSID mis-match, then I would be inclined to treat it like a first class health state like the others - under, over, mis-replicated, unhealthy, QC, QC-Struck, BCSID-mismatch. This would simplify the code a bit and also keep us with the "each container can only have one state at a time" rule. @siddhantsangwan had a lot of experience with RM and @devmadhuu has worked on lot of the Recon state reporting. What do you guys think? -- 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]
