[GitHub] [accumulo] jmark99 commented on a change in pull request #1472: Update ListVolumesUsed to support root table metadata (#1350)

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

jmark99 commented on a change in pull request #1472: Update ListVolumesUsed to support root table metadata (#1350)
URL: https://github.com/apache/accumulo/pull/1472#discussion_r365374359
 
 

 ##########
 File path: server/base/src/main/java/org/apache/accumulo/server/util/ListVolumesUsed.java
 ##########
 @@ -131,7 +113,11 @@ private static void listTable(Ample.DataLevel level, ServerContext context) thro
   }
 
   public static void listVolumes(ServerContext context) throws Exception {
-    listZookeeper(context);
+    try {
+      listTable(Ample.DataLevel.ROOT, context);
+    } catch (UnsupportedOperationException ex) {
+      // print nothing if no table name exists
+    }
 
 Review comment:
   If it's not caught, the command errors out without completing. The previous method just iterated over a collection of 0 elements and continued processing. 

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


With regards,
Apache Git Services