merlimat commented on a change in pull request #1592: Add properties alias for 
tenants in pulsar-admin
URL: https://github.com/apache/incubator-pulsar/pull/1592#discussion_r181909505
 
 

 ##########
 File path: 
pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdTenants.java
 ##########
 @@ -112,4 +112,16 @@ public CmdTenants(PulsarAdmin admin) {
         jcommander.addCommand("delete", new Delete());
     }
 
+    @Parameters(hidden = true)
+    static class CmdProperties extends CmdTenants {
+        public CmdProperties(PulsarAdmin admin) {
+            super(admin);
+        }
+
+        @Override
+        public boolean run(String[] args) {
+            System.err.println("WARN: The properties subcommand is deprecated. 
Please use tenants instead");
 
 Review comment:
   The warning could break scripts if doing `pulsar-admin properties list | 
xargs ... `

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to