From 3c0adac88c3cddb821a732aea02bc068cafd7ea8 Mon Sep 17 00:00:00 2001 From: Andrew Wang Date: Wed, 16 Dec 2015 16:21:06 -0800 Subject: [PATCH] HDFS-9300. TestDirectoryScanner.testThrottle() is still a little flakey. Contributed by Daniel Templeton. --- hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt | 3 +++ .../hadoop/hdfs/server/datanode/TestDirectoryScanner.java | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt index 69fa302eb8d..2d7d1706e6b 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt +++ b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt @@ -901,6 +901,9 @@ Release 2.9.0 - UNRELEASED HDFS-9371. Code cleanup for DatanodeManager. (jing9) + HDFS-9300. TestDirectoryScanner.testThrottle() is still a little flakey. + (Daniel Templeton via wang) + OPTIMIZATIONS BUG FIXES diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/datanode/TestDirectoryScanner.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/datanode/TestDirectoryScanner.java index 250e07d86f2..d030144a583 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/datanode/TestDirectoryScanner.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/datanode/TestDirectoryScanner.java @@ -601,7 +601,7 @@ public void testThrottling() throws Exception { ratio = 0.0f; retries = maxRetries; - while ((retries > 0) && ((ratio < 3f) || (ratio > 4.5f))) { + while ((retries > 0) && ((ratio < 2.75f) || (ratio > 4.5f))) { scanner = new DirectoryScanner(dataNode, fds, conf); ratio = runThrottleTest(blocks); retries -= 1; @@ -610,7 +610,7 @@ public void testThrottling() throws Exception { // Waiting should be about 4x running. LOG.info("RATIO: " + ratio); assertTrue("Throttle is too restrictive", ratio <= 4.5f); - assertTrue("Throttle is too permissive", ratio >= 3.0f); + assertTrue("Throttle is too permissive", ratio >= 2.75f); // Test with more than 1 thread conf.setInt(DFSConfigKeys.DFS_DATANODE_DIRECTORYSCAN_THREADS_KEY, 3);