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/ee725689 Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/ee725689 Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/ee725689 Branch: refs/heads/0.98 Commit: ee725689641d6e61146cb3be77e3f29fa5f54606 Parents: d19b5cd Author: Jonathan M Hsieh <jmhs...@apache.org> Authored: Wed Apr 8 14:45:27 2015 -0700 Committer: Andrew Purtell <apurt...@apache.org> Committed: Mon May 11 13:17:52 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/ee725689/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 a16f531..843fba8 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 @@ -112,7 +112,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