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