Repository: hbase
Updated Branches:
  refs/heads/master 9593a9f39 -> cd2588001


HBASE-15933 Addendum - make merge decision when sizes of both regions are known


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

Branch: refs/heads/master
Commit: cd2588001cf31ad2fb2020f9e021c9b1be1b76fc
Parents: 9593a9f
Author: tedyu <yuzhih...@gmail.com>
Authored: Thu Jun 2 09:30:36 2016 -0700
Committer: tedyu <yuzhih...@gmail.com>
Committed: Thu Jun 2 09:30:36 2016 -0700

----------------------------------------------------------------------
 .../hadoop/hbase/master/normalizer/SimpleRegionNormalizer.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/cd258800/hbase-server/src/main/java/org/apache/hadoop/hbase/master/normalizer/SimpleRegionNormalizer.java
----------------------------------------------------------------------
diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/normalizer/SimpleRegionNormalizer.java
 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/normalizer/SimpleRegionNormalizer.java
index d209eb7..7a54d87 100644
--- 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/normalizer/SimpleRegionNormalizer.java
+++ 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/normalizer/SimpleRegionNormalizer.java
@@ -182,7 +182,7 @@ public class SimpleRegionNormalizer implements 
RegionNormalizer {
         if (mergeEnabled) {
           HRegionInfo hri2 = tableRegions.get(candidateIdx+1);
           long regionSize2 = getRegionSize(hri2);
-          if (regionSize + regionSize2 < avgRegionSize) {
+          if (regionSize > 0 && regionSize2 > 0 && regionSize + regionSize2 < 
avgRegionSize) {
             LOG.info("Table " + table + ", small region size: " + regionSize
               + " plus its neighbor size: " + regionSize2
               + ", less than the avg size " + avgRegionSize + ", merging 
them");

Reply via email to