tengqm commented on code in PR #5988:
URL: https://github.com/apache/gravitino/pull/5988#discussion_r1898445314
##########
clients/cli/src/main/java/org/apache/gravitino/cli/GravitinoCommandLine.java:
##########
@@ -762,6 +775,12 @@ protected void handleRoleCommand() {
}
}
+ private String getOneRole(String[] roles, String command) {
+ Preconditions.checkArgument(
+ roles.length == 1, command + " requires only one role, but multiple
are currently passed.");
Review Comment:
Got it. But this is weird.
The command line will automagically interpret everything that appear after
`--role` as the argument value for `--role`?
I was thinking that if you want to specify multiple values for a single
command, the values must be a comma-separated string like this, `--role
roleA,roleB`.
Maybe I'm wrong.
--
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]
