ctubbsii commented on code in PR #2987:
URL: https://github.com/apache/accumulo/pull/2987#discussion_r984815788
##########
server/base/src/main/java/org/apache/accumulo/server/util/Admin.java:
##########
@@ -250,7 +251,7 @@ static class FateOpsCommand {
boolean printJson;
@Parameter(names = {"-s", "--state"},
- description = "<state>[ -s <state>...] Print transactions in the
state(s) {NEW, IN_PROGRESS, FAILED_IN_PROGRESS, FAILED, SUCCESSFUL}")
+ description = "<state>... Print transactions in the state(s) {NEW,
IN_PROGRESS, FAILED_IN_PROGRESS, FAILED, SUCCESSFUL}")
Review Comment:
Okay. I don't know that we need to be tied to that same flag in the new
tool, as long as we preserve all the same functionality, and filtering
functionality is still easy and convenient, regardless of whether we carry over
this flag. I'm not a big fan of carrying it over, but I'm not strongly opposed
either... I just think it's unnecessary and redundant, and I'd prefer to delete
it.
--
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]
