[GitHub] ivankelly commented on a change in pull request #1592: Add properties alias for tenants in pulsar-admin

[GitBox <gi...@apache.org>]

ivankelly 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_r181912276
 
 

 ##########
 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:
   Shouldn't do unless they do 2>&1, since it's going to stderr

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services