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/9629f31c
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/9629f31c
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/9629f31c

Branch: refs/heads/HBASE-15631-branch-1
Commit: 9629f31c0eb9eab6aa4476e3461d0e5a82b5bbf7
Parents: 3a98817
Author: Andrew Purtell <apurt...@apache.org>
Authored: Wed Jul 5 15:37:10 2017 -0700
Committer: Andrew Purtell <apurt...@apache.org>
Committed: Wed Jul 19 16:21:10 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/9629f31c/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