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

busbey pushed a commit to branch branch-2
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2 by this push:
     new 7e04eeb  HBASE-23040 give friendly message about unknown server when 
running unload for RegionMover.
7e04eeb is described below

commit 7e04eebc6225d6d36785b7e71b142bf1fcca8785
Author: Sean Busbey <bus...@apache.org>
AuthorDate: Tue Sep 17 22:47:17 2019 -0500

    HBASE-23040 give friendly message about unknown server when running unload 
for RegionMover.
    
    closes #635
    
    Signed-off-by: stack <st...@apache.org>
    (cherry picked from commit ab076b07313fe7b86a010eca38ac8db7de405969)
---
 .../src/main/java/org/apache/hadoop/hbase/util/RegionMover.java     | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/util/RegionMover.java 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/util/RegionMover.java
index fdd079e..0107196 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/util/RegionMover.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/util/RegionMover.java
@@ -421,6 +421,12 @@ public class RegionMover extends AbstractHBaseTool 
implements Closeable {
                 .keySet());
         // Remove the host Region server from target Region Servers list
         ServerName server = stripServer(regionServers, hostname, port);
+        if (server == null) {
+          LOG.info("Could not find server '{}:{}' in the set of region 
servers. giving up.",
+              hostname, port);
+          LOG.debug("List of region servers: {}", regionServers);
+          return false;
+        }
         // Remove RS present in the exclude file
         stripExcludes(regionServers);
         stripMaster(regionServers);

Reply via email to