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/07/19 00:45:01 UTC
[19/21] 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
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/a0783e32
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/a0783e32
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/a0783e32
Branch: refs/heads/HBASE-15631-branch-1
Commit: a0783e326fc84b59219e68342fe6badf1cddd538
Parents: 4a969b2
Author: Andrew Purtell <ap...@apache.org>
Authored: Mon Jul 3 17:54:36 2017 -0700
Committer: Andrew Purtell <ap...@apache.org>
Committed: Tue Jul 18 17:44:32 2017 -0700
----------------------------------------------------------------------
.../main/java/org/apache/hadoop/hbase/master/LoadBalancer.java | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/a0783e32/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 937b32f..a80cdc3 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,8 +52,9 @@ import org.apache.hadoop.hbase.TableName;
@InterfaceAudience.Private
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.parseServerName("localhost,1,1");
+ // Used to signal to the caller that the region(s) cannot be assigned
+ // 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