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 15:34:05 UTC

[hbase] branch branch-2.1 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 branch-2.1
in repository https://gitbox.apache.org/repos/asf/hbase.git


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

commit 39222cae57faebfeed667017a9c447b857d3eb1a
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>
    (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 70a4f0e..3fbbc19 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);