You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ap...@apache.org on 2017/07/19 00:44:55 UTC

[13/21] hbase git commit: HBASE-15848 Fix possible null point dereference in RSGroupBasedLoadBalancer#getMisplacedRegions (Stephen Yuan Jiang)

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 <ap...@apache.org>
Authored: Wed Jul 5 15:37:10 2017 -0700
Committer: Andrew Purtell <ap...@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);
         }