You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by jm...@apache.org on 2015/04/08 23:53:05 UTC

hbase git commit: HBASE-13412 ADDENDUM Region split decisions should have jitter

Repository: hbase
Updated Branches:
  refs/heads/hbase-1 [created] c2bfddac1


HBASE-13412 ADDENDUM Region split decisions should have jitter

This corrects the jitter threshold constant in the unit test so that it will pass regularly.


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

Branch: refs/heads/hbase-1
Commit: c2bfddac136008ca1be00031aaf83f6f35792476
Parents: 408b916
Author: Jonathan M Hsieh <jm...@apache.org>
Authored: Wed Apr 8 14:45:27 2015 -0700
Committer: Jonathan M Hsieh <jm...@apache.org>
Committed: Wed Apr 8 14:50:03 2015 -0700

----------------------------------------------------------------------
 .../apache/hadoop/hbase/regionserver/TestRegionSplitPolicy.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/c2bfddac/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionSplitPolicy.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionSplitPolicy.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionSplitPolicy.java
index 3146635..3b29a46 100644
--- a/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionSplitPolicy.java
+++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestRegionSplitPolicy.java
@@ -110,7 +110,7 @@ public class TestRegionSplitPolicy {
     regions.add(mockRegion);
     assertFalse(policy.shouldSplit());
     // make sure its just over; verify it'll split
-    Mockito.doReturn((long)(maxSplitSize * 1.025 + 1)).when(mockStore).getSize();
+    Mockito.doReturn((long)(maxSplitSize * 1.25 + 1)).when(mockStore).getSize();
     assertTrue(policy.shouldSplit());
 
     // Finally assert that even if loads of regions, we'll split at max size