szetszwo commented on PR #9472: URL: https://github.com/apache/ozone/pull/9472#issuecomment-6433217038
@devmadhuu , thanks for the explaination! > I propose a small follow-up PR to remove the numeric field, ... It sounds great. Please do the follow-up. Since it is for CLI report and metrics, let's also rename the ContainerHealthState enum to something like HealthStateMetricName. > **QUASI_CLOSED_STUCK_UNDER_REPLICATED** - A container cannot safely finish closing, and the check for stuck quasi-closed containers also finds too few copies of the remaining original replicas. But why there are no QUASI_CLOSED without STUCK? The current enum definitions are non-precise and incomplete. We need to come up with some precise definitions first. -- 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]
