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