You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by wa...@apache.org on 2016/01/25 13:00:59 UTC
hadoop git commit: HADOOP-12715.
TestValueQueue#testgetAtMostPolicyALL fails intermittently. Contributed by
Xiao Chen.
Repository: hadoop
Updated Branches:
refs/heads/trunk 56a0c1750 -> 6eacdea0e
HADOOP-12715. TestValueQueue#testgetAtMostPolicyALL fails intermittently. Contributed by Xiao Chen.
Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/6eacdea0
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/6eacdea0
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/6eacdea0
Branch: refs/heads/trunk
Commit: 6eacdea0e475b4fff91cedce5005a7c11749cf64
Parents: 56a0c17
Author: Walter Su <wa...@apache.org>
Authored: Mon Jan 25 19:30:04 2016 +0800
Committer: Walter Su <wa...@apache.org>
Committed: Mon Jan 25 19:30:04 2016 +0800
----------------------------------------------------------------------
hadoop-common-project/hadoop-common/CHANGES.txt | 3 +
.../hadoop/crypto/key/TestValueQueue.java | 65 +++++++++++++++-----
2 files changed, 52 insertions(+), 16 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hadoop/blob/6eacdea0/hadoop-common-project/hadoop-common/CHANGES.txt
----------------------------------------------------------------------
diff --git a/hadoop-common-project/hadoop-common/CHANGES.txt b/hadoop-common-project/hadoop-common/CHANGES.txt
index 2fc8ab4..9606296 100644
--- a/hadoop-common-project/hadoop-common/CHANGES.txt
+++ b/hadoop-common-project/hadoop-common/CHANGES.txt
@@ -2496,6 +2496,9 @@ Release 2.6.4 - UNRELEASED
HADOOP-12736. TestTimedOutTestsListener#testThreadDumpAndDeadlocks
sometimes times out. (Xiao Chen via aajisaka)
+ HADOOP-12715. TestValueQueue#testgetAtMostPolicyALL fails intermittently.
+ (Xiao Chen via waltersu4549)
+
Release 2.6.3 - 2015-12-17
INCOMPATIBLE CHANGES
http://git-wip-us.apache.org/repos/asf/hadoop/blob/6eacdea0/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/crypto/key/TestValueQueue.java
----------------------------------------------------------------------
diff --git a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/crypto/key/TestValueQueue.java b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/crypto/key/TestValueQueue.java
index 8e3a093..5eae9a0 100644
--- a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/crypto/key/TestValueQueue.java
+++ b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/crypto/key/TestValueQueue.java
@@ -19,18 +19,24 @@ package org.apache.hadoop.crypto.key;
import java.io.IOException;
import java.util.Queue;
+import java.util.concurrent.ExecutionException;
import java.util.concurrent.LinkedBlockingQueue;
import java.util.concurrent.TimeUnit;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
import org.apache.hadoop.crypto.key.kms.ValueQueue;
import org.apache.hadoop.crypto.key.kms.ValueQueue.QueueRefiller;
import org.apache.hadoop.crypto.key.kms.ValueQueue.SyncGenerationPolicy;
+import org.apache.hadoop.test.GenericTestUtils;
import org.junit.Assert;
import org.junit.Test;
+import com.google.common.base.Supplier;
import com.google.common.collect.Sets;
public class TestValueQueue {
+ Logger LOG = LoggerFactory.getLogger(TestValueQueue.class);
private static class FillInfo {
final int num;
@@ -60,7 +66,7 @@ public class TestValueQueue {
/**
* Verifies that Queue is initially filled to "numInitValues"
*/
- @Test
+ @Test(timeout=30000)
public void testInitFill() throws Exception {
MockFiller filler = new MockFiller();
ValueQueue<String> vq =
@@ -74,7 +80,7 @@ public class TestValueQueue {
/**
* Verifies that Queue is initialized (Warmed-up) for provided keys
*/
- @Test
+ @Test(timeout=30000)
public void testWarmUp() throws Exception {
MockFiller filler = new MockFiller();
ValueQueue<String> vq =
@@ -97,7 +103,7 @@ public class TestValueQueue {
* Verifies that the refill task is executed after "checkInterval" if
* num values below "lowWatermark"
*/
- @Test
+ @Test(timeout=30000)
public void testRefill() throws Exception {
MockFiller filler = new MockFiller();
ValueQueue<String> vq =
@@ -116,7 +122,7 @@ public class TestValueQueue {
* Verifies that the No refill Happens after "checkInterval" if
* num values above "lowWatermark"
*/
- @Test
+ @Test(timeout=30000)
public void testNoRefill() throws Exception {
MockFiller filler = new MockFiller();
ValueQueue<String> vq =
@@ -131,29 +137,56 @@ public class TestValueQueue {
/**
* Verify getAtMost when SyncGeneration Policy = ALL
*/
- @Test
+ @Test(timeout=30000)
public void testgetAtMostPolicyALL() throws Exception {
MockFiller filler = new MockFiller();
- ValueQueue<String> vq =
+ final ValueQueue<String> vq =
new ValueQueue<String>(10, 0.1f, 300, 1,
SyncGenerationPolicy.ALL, filler);
Assert.assertEquals("test", vq.getNext("k1"));
Assert.assertEquals(1, filler.getTop().num);
- // Drain completely
- Assert.assertEquals(10, vq.getAtMost("k1", 10).size());
- // Synchronous call
- Assert.assertEquals(10, filler.getTop().num);
- // Ask for more... return all
- Assert.assertEquals(19, vq.getAtMost("k1", 19).size());
+
+ // Synchronous call:
+ // 1. Synchronously fill returned list
+ // 2. Start another async task to fill the queue in the cache
+ Assert.assertEquals("Failed in sync call.", 10,
+ vq.getAtMost("k1", 10).size());
+ Assert.assertEquals("Sync call filler got wrong number.", 10,
+ filler.getTop().num);
+
+ // Wait for the async task to finish
+ GenericTestUtils.waitFor(new Supplier<Boolean>() {
+ @Override
+ public Boolean get() {
+ try {
+ int size = vq.getSize("k1");
+ if (size != 10) {
+ LOG.info("Current ValueQueue size is " + size);
+ return false;
+ }
+ return true;
+ } catch (ExecutionException e) {
+ LOG.error("Exception when getSize.", e);
+ return false;
+ }
+ }
+ }, 100, 3000);
+ Assert.assertEquals("Failed in async call.", 10, filler.getTop().num);
+
+ // Drain completely after filled by the async thread
+ Assert.assertEquals("Failed to drain completely after async.", 10,
+ vq.getAtMost("k1", 10).size());
// Synchronous call (No Async call since num > lowWatermark)
- Assert.assertEquals(19, filler.getTop().num);
+ Assert.assertEquals("Failed to get all 19.", 19,
+ vq.getAtMost("k1", 19).size());
+ Assert.assertEquals("Failed in sync call.", 19, filler.getTop().num);
vq.shutdown();
}
/**
* Verify getAtMost when SyncGeneration Policy = ALL
*/
- @Test
+ @Test(timeout=30000)
public void testgetAtMostPolicyATLEAST_ONE() throws Exception {
MockFiller filler = new MockFiller();
ValueQueue<String> vq =
@@ -171,7 +204,7 @@ public class TestValueQueue {
/**
* Verify getAtMost when SyncGeneration Policy = LOW_WATERMARK
*/
- @Test
+ @Test(timeout=30000)
public void testgetAtMostPolicyLOW_WATERMARK() throws Exception {
MockFiller filler = new MockFiller();
ValueQueue<String> vq =
@@ -188,7 +221,7 @@ public class TestValueQueue {
vq.shutdown();
}
- @Test
+ @Test(timeout=30000)
public void testDrain() throws Exception {
MockFiller filler = new MockFiller();
ValueQueue<String> vq =