You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ap...@apache.org on 2017/06/21 21:50:56 UTC
[2/2] hbase git commit: HBASE-18235 LoadBalancer.BOGUS_SERVER_NAME
should not have a bogus hostname
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 <ap...@apache.org>
Authored: Wed Jun 21 14:37:47 2017 -0700
Committer: Andrew Purtell <ap...@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