hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From apurt...@apache.org
Subject [1/2] hbase git commit: HBASE-18235 LoadBalancer.BOGUS_SERVER_NAME should not have a bogus hostname
Date Wed, 21 Jun 2017 21:50:55 GMT
Repository: hbase
Updated Branches:
  refs/heads/branch-2 be559f1eb -> ff5d49731
  refs/heads/master d8e0163fa -> 3489a1b82


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/3489a1b8
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/3489a1b8
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/3489a1b8

Branch: refs/heads/master
Commit: 3489a1b82144d2481e0b08f8d898c0e2a5c24623
Parents: d8e0163
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:37:47 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/3489a1b8/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