ayushtkn commented on code in PR #11015:
URL: https://github.com/apache/ozone/pull/11015#discussion_r3776721598


##########
hadoop-hdds/docs/content/tools/debug/DebugReplicas.md:
##########
@@ -109,6 +110,13 @@ failed checks.
       --container-state   Check the container and replica states. Containers in
                             [DELETING, DELETED] states, or it's replicas in
                             [DELETED, UNHEALTHY, INVALID] states fail the 
check.
+      --refresh-container-locations-from-scm
+                          Force OM to refresh container locations from SCM
+                            before returning key info. This can slow down
+                            verification but may avoid stale location results
+                            from OM cache.
+      --refresh-from-scm  Short alias for
+                            --refresh-container-locations-from-scm.

Review Comment:
   I don't catch this, why do we need two names? Why we can't just have one 
maybe the shorter one, make sure its defination is well, rather than having two 
and having a short alias for one command. That too a debug command



-- 
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]

Reply via email to