From 70732ead891c93be01bddbc7b49003c1d582c979 Mon Sep 17 00:00:00 2001 From: c00443737 Date: Thu, 11 Jun 2020 11:25:53 +0800 Subject: [PATCH] [HBASE-24395]ServerName#getHostname() is case sensitive --- .../org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java index ea9c980..8a5a0e9 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java @@ -1585,7 +1585,7 @@ public abstract class BaseLoadBalancer implements LoadBalancer { ArrayListMultimap serversByHostname = ArrayListMultimap.create(); for (ServerName server : servers) { assignments.put(server, new ArrayList()); - serversByHostname.put(server.getHostname(), server); + serversByHostname.put(server.getHostname().toLowerCase(), server); } // Collection of the hostnames that used to have regions @@ -1604,13 +1604,13 @@ public abstract class BaseLoadBalancer implements LoadBalancer { ServerName oldServerName = entry.getValue(); List localServers = new ArrayList(); if (oldServerName != null) { - localServers = serversByHostname.get(oldServerName.getHostname()); + localServers = serversByHostname.get(oldServerName.getHostname().toLowerCase()); } if (localServers.isEmpty()) { // No servers on the new cluster match up with this hostname, assign randomly, later. randomAssignRegions.add(region); if (oldServerName != null) { - oldHostsNoLongerPresent.add(oldServerName.getHostname()); + oldHostsNoLongerPresent.add(oldServerName.getHostname().toLowerCase()); } } else if (localServers.size() == 1) { // the usual case - one new server on same host -- 2.9.2.windows.1