codelipenghui commented on a change in pull request #9750:
URL: https://github.com/apache/pulsar/pull/9750#discussion_r584383280
##########
File path:
pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdNamespaces.java
##########
@@ -382,7 +382,7 @@ void run() throws PulsarAdminException {
@Parameter(description = "tenant/namespace", required = true)
private java.util.List<String> params;
- @Parameter(names = { "--maxSubscriptionsPerTopic", "-m" }, description
= "Max subscriptions per topic",
+ @Parameter(names = { "--max-subscriptions-per-topic", "-m" },
description = "Max subscriptions per topic",
Review comment:
Could you please deprecate the old parameter name instead of deleting it
directly? We can make the old one does not appear in the CLI help but also able
to use it. Then we can try to delete it in the 2.8.0 or later.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
