HBASE-15848 Fix possible null point dereference in 
RSGroupBasedLoadBalancer#getMisplacedRegions (Stephen Yuan Jiang)


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/aa8a1b84
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/aa8a1b84
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/aa8a1b84

Branch: refs/heads/HBASE-15631-branch-1
Commit: aa8a1b84cb8ba26be96c67dbdf149196cbdd2721
Parents: c810511
Author: Andrew Purtell <apurt...@apache.org>
Authored: Wed Jul 5 15:37:10 2017 -0700
Committer: Andrew Purtell <apurt...@apache.org>
Committed: Tue Jul 18 17:44:31 2017 -0700

----------------------------------------------------------------------
 .../apache/hadoop/hbase/rsgroup/RSGroupBasedLoadBalancer.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/aa8a1b84/hbase-rsgroup/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupBasedLoadBalancer.java
----------------------------------------------------------------------
diff --git 
a/hbase-rsgroup/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupBasedLoadBalancer.java
 
b/hbase-rsgroup/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupBasedLoadBalancer.java
index fea1275..f69f093 100644
--- 
a/hbase-rsgroup/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupBasedLoadBalancer.java
+++ 
b/hbase-rsgroup/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupBasedLoadBalancer.java
@@ -331,7 +331,7 @@ public class RSGroupBasedLoadBalancer implements 
RSGroupableBalancer, LoadBalanc
             " on server: " + assignedServer +
             " found in group: " +
             
RSGroupInfoManager.getRSGroupOfServer(assignedServer.getHostPort()) +
-            " outside of group: " + info.getName());
+            " outside of group: " + (info == null ? "UNKNOWN" : 
info.getName()));
         misplacedRegions.add(region);
       }
     }
@@ -352,7 +352,7 @@ public class RSGroupBasedLoadBalancer implements 
RSGroupableBalancer, LoadBalanc
         try {
           info = RSGroupInfoManager.getRSGroup(
               RSGroupInfoManager.getRSGroupOfTable(region.getTable()));
-        }catch(IOException exp){
+        } catch (IOException exp) {
           LOG.debug("Group information null for region of table " + 
region.getTable(),
               exp);
         }

Reply via email to