[pulsar-cli] fix ns-isolation-policy api output (apache#5313)

tahirh · Oct 7, 2019 · de233d4 · de233d4
1 parent e840375
commit de233d4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 12 deletions.
diff --git a/pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CliCommand.java b/pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CliCommand.java
@@ -171,15 +171,7 @@ static Set<AuthAction> getAuthActions(List<String> actions) {
 
     <T> void print(List<T> items) {
         for (T item : items) {
-            System.out.println(item);
-        }
-    }
-
-    <T> void printList(T item) {
-        try {
-            System.out.println(writer.writeValueAsString(item));
-        } catch (Exception e) {
-            throw new RuntimeException(e);
+            print(item);
         }
     }
 

diff --git a/pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdFunctionWorker.java b/pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdFunctionWorker.java
@@ -50,7 +50,7 @@ class FunctionsStats extends BaseCommand {
 
         @Override
         void runCmd() throws Exception {
-            printList(admin.worker().getFunctionsStats());
+            print(admin.worker().getFunctionsStats());
         }
     }
 
@@ -59,7 +59,7 @@ class CmdMonitoringMetrics extends BaseCommand {
 
         @Override
         void runCmd() throws Exception {
-            printList(admin.worker().getMetrics());
+            print(admin.worker().getMetrics());
         }
     }
 
@@ -68,7 +68,7 @@ class GetCluster extends BaseCommand {
 
         @Override
         void runCmd() throws Exception {
-            printList(admin.worker().getCluster());
+            print(admin.worker().getCluster());
         }
     }