diff --git a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt index 19260b561df..dc7e02c7b21 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt +++ b/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt @@ -288,6 +288,9 @@ Trunk (Unreleased) HDFS-4275. MiniDFSCluster-based tests fail on Windows due to failure to delete test namenode directory. (Chris Nauroth via suresh) + HDFS-4338. TestNameNodeMetrics#testCorruptBlock is flaky. (Andrew Wang via + atm) + Release 2.0.3-alpha - Unreleased INCOMPATIBLE CHANGES diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/metrics/TestNameNodeMetrics.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/metrics/TestNameNodeMetrics.java index 96a9966917e..d31f12ddfa3 100644 --- a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/metrics/TestNameNodeMetrics.java +++ b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/metrics/TestNameNodeMetrics.java @@ -222,11 +222,9 @@ public class TestNameNodeMetrics { cluster.getNamesystem().writeUnlock(); } BlockManagerTestUtil.getComputedDatanodeWork(bm); - BlockManagerTestUtil.updateState(bm); MetricsRecordBuilder rb = getMetrics(NS_METRICS); assertGauge("CorruptBlocks", 1L, rb); assertGauge("PendingReplicationBlocks", 1L, rb); - assertGauge("ScheduledReplicationBlocks", 1L, rb); fs.delete(file, true); // During the file deletion, both BlockManager#corruptReplicas and @@ -235,7 +233,7 @@ public class TestNameNodeMetrics { // corruptReplicas and pendingReplications. The corresponding // metrics (CorruptBlocks and PendingReplicationBlocks) will only be updated // when BlockManager#computeDatanodeWork is run where the - // BlockManager#udpateState is called. And in + // BlockManager#updateState is called. And in // BlockManager#computeDatanodeWork the metric ScheduledReplicationBlocks // will also be updated. rb = waitForDnMetricValue(NS_METRICS, "CorruptBlocks", 0L);