[accumulo] branch main updated: Log error in Admin for no servers (#2748)

[mmiller]

This is an automated email from the ASF dual-hosted git repository.

mmiller pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/main by this push:
     new 62aa21e40e Log error in Admin for no servers (#2748)
62aa21e40e is described below

commit 62aa21e40ebdcfd0a3c1d864d6adfe2c3de2de27
Author: Mike Miller <mmil...@apache.org>
AuthorDate: Tue May 31 16:08:07 2022 +0000

    Log error in Admin for no servers (#2748)
---
 .../base/src/main/java/org/apache/accumulo/server/util/Admin.java   | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git 
a/server/base/src/main/java/org/apache/accumulo/server/util/Admin.java 
b/server/base/src/main/java/org/apache/accumulo/server/util/Admin.java
index 7ebefc472e..912c5113c8 100644
--- a/server/base/src/main/java/org/apache/accumulo/server/util/Admin.java
+++ b/server/base/src/main/java/org/apache/accumulo/server/util/Admin.java
@@ -115,7 +115,7 @@ public class Admin implements KeywordExecutable {
   @Parameters(commandDescription = "stop the master (DEPRECATED -- use 
stopManager instead)")
   static class StopMasterCommand {}
 
-  @Parameters(commandDescription = "stop all the servers")
+  @Parameters(commandDescription = "stop all of the tablets servers and 
manager")
   static class StopAllCommand {}
 
   @Parameters(commandDescription = "list Accumulo instances in zookeeper")
@@ -379,6 +379,10 @@ public class Admin implements KeywordExecutable {
       log.info("No managers running. Not attempting safe unload of tserver.");
       return;
     }
+    if (servers.isEmpty()) {
+      log.error("No tablet servers provided.");
+      return;
+    }
 
     final String zTServerRoot = getTServersZkPath(context);
     final ZooCache zc = context.getZooCache();