HBASE-3175 Commit of HBASE-3160 broke TestPriorityCompactionQueue up on hudson (nicolas via jgray)
git-svn-id: https://svn.apache.org/repos/asf/hbase/trunk@1029105 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
c9da74ebc7
commit
93588c8cfc
|
@ -627,6 +627,8 @@ Release 0.21.0 - Unreleased
|
|||
HBASE-3172 Reverse order of AssignmentManager and MetaNodeTracker in
|
||||
ZooKeeperWatcher
|
||||
HBASE-2406 Define semantics of cell timestamps/versions
|
||||
HBASE-3175 Commit of HBASE-3160 broke TestPriorityCompactionQueue up on
|
||||
hudson (nicolas via jgray)
|
||||
|
||||
|
||||
IMPROVEMENTS
|
||||
|
|
|
@ -104,11 +104,11 @@ public class TestPriorityCompactionQueue {
|
|||
|
||||
// test 1
|
||||
// check fifo w/priority
|
||||
addRegion(pq, r1, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r2, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r3, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r4, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r5, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r1, 0);
|
||||
addRegion(pq, r2, 0);
|
||||
addRegion(pq, r3, 0);
|
||||
addRegion(pq, r4, 0);
|
||||
addRegion(pq, r5, 0);
|
||||
|
||||
getAndCheckRegion(pq, r1);
|
||||
getAndCheckRegion(pq, r2);
|
||||
|
@ -118,11 +118,11 @@ public class TestPriorityCompactionQueue {
|
|||
|
||||
// test 2
|
||||
// check fifo w/mixed priority
|
||||
addRegion(pq, r1, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r1, 0);
|
||||
addRegion(pq, r2, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r3, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r3, 0);
|
||||
addRegion(pq, r4, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r5, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r5, 0);
|
||||
|
||||
getAndCheckRegion(pq, r1);
|
||||
getAndCheckRegion(pq, r3);
|
||||
|
@ -136,7 +136,7 @@ public class TestPriorityCompactionQueue {
|
|||
addRegion(pq, r2, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r3, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r4, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r5, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r5, 0);
|
||||
|
||||
getAndCheckRegion(pq, r5);
|
||||
getAndCheckRegion(pq, r1);
|
||||
|
@ -147,11 +147,11 @@ public class TestPriorityCompactionQueue {
|
|||
// test 4
|
||||
// check fifo w/mixed priority elevation time
|
||||
addRegion(pq, r1, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r2, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r2, 0);
|
||||
addRegion(pq, r3, CompactSplitThread.PRIORITY_USER);
|
||||
Thread.sleep(1000);
|
||||
addRegion(pq, r4, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r5, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r5, 0);
|
||||
|
||||
getAndCheckRegion(pq, r2);
|
||||
getAndCheckRegion(pq, r5);
|
||||
|
@ -170,7 +170,7 @@ public class TestPriorityCompactionQueue {
|
|||
addRegion(pq, r3, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r4, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r5, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r3, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r3, 0);
|
||||
|
||||
getAndCheckRegion(pq, r3);
|
||||
getAndCheckRegion(pq, r1);
|
||||
|
@ -205,14 +205,14 @@ public class TestPriorityCompactionQueue {
|
|||
// we can handle negative priorities
|
||||
addRegion(pq, r1, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r2, -1);
|
||||
addRegion(pq, r3, CompactSplitThread.PRIORITY_USER);
|
||||
addRegion(pq, r3, 0);
|
||||
addRegion(pq, r4, -2);
|
||||
|
||||
|
||||
getAndCheckRegion(pq, r4);
|
||||
getAndCheckRegion(pq, r2);
|
||||
getAndCheckRegion(pq, r3);
|
||||
getAndCheckRegion(pq, r1);
|
||||
|
||||
|
||||
Assert.assertTrue("Queue should be empty.", pq.size() == 0);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue