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/f7c37893 Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/f7c37893 Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/f7c37893 Branch: refs/heads/HBASE-15631-branch-1 Commit: f7c378935474e6c39b1b2dfb3c9520e3ebf17d60 Parents: 56baade Author: Andrew Purtell <[email protected]> Authored: Mon Jul 3 17:54:36 2017 -0700 Committer: Andrew Purtell <[email protected]> Committed: Thu Jul 20 17:55:42 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/f7c37893/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
