dlmarion commented on a change in pull request #2238:
URL: https://github.com/apache/accumulo/pull/2238#discussion_r694060387
##########
File path: server/base/src/main/java/org/apache/accumulo/server/util/ZooZap.java
##########
@@ -61,6 +61,10 @@ private static void message(String msg, Opts opts) {
boolean zapTservers = false;
@Parameter(names = "-tracers", description = "remove tracer locks")
boolean zapTracers = false;
+ @Parameter(names = "-coordinators", description = "remove compaction
coordinator locks")
Review comment:
> I wish compaction-coordinators wasn't so lengthy. -coordinators is
just so generic, when we're referring to a specific coordinator.
In the context of ZooZap, it's designed to clean up things in ZooKeeper for
all servers of a given type. I don't think you can use ZooZap to clean up
entries for a specific tserver. Regarding the parameter name, we don't have
anything else called a coordinator today. I'm not sure where the confusion
would come to play.
--
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]
