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 <[email protected]> Authored: Wed Jun 21 14:37:47 2017 -0700 Committer: Andrew Purtell <[email protected]> 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
