You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by bu...@apache.org on 2019/09/18 11:28:31 UTC

[hbase] branch master updated: HBASE-23040 give friendly message about unknown server when running unload for RegionMover.

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

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


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

commit ab076b07313fe7b86a010eca38ac8db7de405969
Author: Sean Busbey <bu...@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>
---
 .../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);