netudima commented on code in PR #2497:
URL: https://github.com/apache/cassandra/pull/2497#discussion_r2094155120


##########
src/java/org/apache/cassandra/tools/nodetool/AccordAdmin.java:
##########
@@ -42,9 +57,9 @@ protected void execute(NodeProbe probe)
     }
 
     @Command(name = "mark_stale", description = "Mark a replica as being stale 
and no longer able to participate in durability status coordination")
-    public static class MarkStale extends AccordAdmin
+    public static class MarkStale extends AbstractCommand
     {
-        @Arguments(required = true, description = "One or more node IDs to 
mark stale", usage = "<nodeId>+")
+        @Parameters(arity = "1", description = "One or more node IDs to mark 
stale")

Review Comment:
   should it be arity = "1..*"?



-- 
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: pr-unsubscr...@cassandra.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org

Reply via email to