hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From apurt...@apache.org
Subject [2/2] hbase git commit: HBASE-18235 LoadBalancer.BOGUS_SERVER_NAME should not have a bogus hostname
Date Wed, 21 Jun 2017 21:50:56 GMT
HBASE-18235 LoadBalancer.BOGUS_SERVER_NAME should not have a bogus hostname

We deliberately use 'localhost' instead of a bogus hostname for
LoadBalancer.BOGUS_SERVER_NAME so the operation will fail fast.


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/ff5d4973
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/ff5d4973
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/ff5d4973

Branch: refs/heads/branch-2
Commit: ff5d497310c5551698acdd8592f4a3e484855d8d
Parents: be559f1
Author: Andrew Purtell <apurtell@apache.org>
Authored: Wed Jun 21 14:37:47 2017 -0700
Committer: Andrew Purtell <apurtell@apache.org>
Committed: Wed Jun 21 14:38:38 2017 -0700

----------------------------------------------------------------------
 .../main/java/org/apache/hadoop/hbase/master/LoadBalancer.java    | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/ff5d4973/hbase-server/src/main/java/org/apache/hadoop/hbase/master/LoadBalancer.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/LoadBalancer.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/LoadBalancer.java
index 4611982..5025566 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/LoadBalancer.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/LoadBalancer.java
@@ -52,7 +52,8 @@ import edu.umd.cs.findbugs.annotations.Nullable;
 public interface LoadBalancer extends Configurable, Stoppable, ConfigurationObserver {
 
   // Used to signal to the caller that the region(s) cannot be assigned
-  ServerName BOGUS_SERVER_NAME = ServerName.valueOf("bogus.example.com,1,1");
+  // We deliberately use 'localhost' so the operation will fail fast
+  ServerName BOGUS_SERVER_NAME = ServerName.valueOf("localhost,1,1");
 
   /**
    * Set the current cluster status.  This allows a LoadBalancer to map host name to a server


Mime
View raw message