0lai0 commented on code in PR #9135:
URL: https://github.com/apache/ozone/pull/9135#discussion_r2588114786
##########
hadoop-ozone/tools/src/main/java/org/apache/hadoop/ozone/debug/replicas/ContainerStateVerifier.java:
##########
@@ -155,18 +166,57 @@ private ContainerInfoToken getContainerInfoToken(long
containerId)
// Cache miss - fetch and store
ContainerInfo info = containerOperationClient.getContainer(containerId);
String encodeToken =
containerOperationClient.getEncodedContainerToken(containerId);
- cachedData = new ContainerInfoToken(info.getState(), encodeToken);
+ cachedData = new ContainerInfoToken(info.getState(), encodeToken, info);
encodedTokenCache.put(containerId, cachedData);
return cachedData;
}
+ private String checkReplicationStatus(long containerId) {
+ try {
+ ContainerInfoToken token = getContainerInfoToken(containerId);
+ ContainerInfo containerInfo = token.getContainerInfo();
+
+ // Get container replicas from SCM
+ List<ContainerReplicaInfo> replicaInfos =
+ containerOperationClient.getContainerReplicas(containerId);
+
+ if (replicaInfos.isEmpty()) {
+ return ContainerHealthResult.ReplicationStatus.UNDER_REPLICATED + ":
no replicas found";
+ }
+
+ int replicationFactor = containerInfo.getReplicationFactor().getNumber();
Review Comment:
Thanks for the pointer. I'll change to use ReplicationConfig.
--
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]
