aryangupta1998 commented on code in PR #11015:
URL: https://github.com/apache/ozone/pull/11015#discussion_r3778361782
##########
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 originally kept two names to balance:
- discoverability/self-documentation (--refresh-container-locations-from-scm)
- typing convenience (--refresh-from-scm)
But this is a debug-only command, so I agree we can keep it simpler and
avoid extra surface area. I’m happy to switch to a single flag and keep the
description explicit so intent is still clear.
If you’re good with it, I’ll retain only --refresh-from-scm and update
help/docs accordingly.
--
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]
