wangzhiwubigdata commented on a change in pull request #14254:
URL: https://github.com/apache/pulsar/pull/14254#discussion_r805113678
##########
File path:
pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdNamespaces.java
##########
@@ -274,11 +274,11 @@ void run() throws PulsarAdminException {
@Parameter(description = "tenant/namespace", required = true)
private java.util.List<String> params;
- @Parameter(names = "--subscription", description = "Subscription name
for which permission "
+ @Parameter(names = {"-s", "--subscription"}, description =
"Subscription name for which permission "
+ "will be revoked to roles", required = true)
private String subscription;
- @Parameter(names = "--role", description = "Client role to which
revoke permissions", required = true)
+ @Parameter(names = {"-r", "--roles"}, description = "Client role to
which revoke permissions", required = true)
Review comment:
> Please keep the previous full name “--role”
sure，fixed
--
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]
