[ https://issues.apache.org/jira/browse/HDFS-17260?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17792342#comment-17792342 ]
ASF GitHub Bot commented on HDFS-17260: --------------------------------------- ayushtkn commented on code in PR #6279: URL: https://github.com/apache/hadoop/pull/6279#discussion_r1412752208 ########## hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/blockmanagement/TestReplicationPolicyExcludeSlowNodes.java: ########## @@ -136,4 +139,40 @@ public void testChooseTargetExcludeSlowNodes() throws Exception { NameNode.LOG.info("Done working on it"); } + @Test + public void testSlowPeerTrackerEnabledClearSlowNodes() throws Exception { + namenode.getNamesystem().writeLock(); + try { + // add nodes + for (DatanodeDescriptor dataNode : dataNodes) { + dnManager.addDatanode(dataNode); + } + + // mock slow nodes + SlowPeerTracker tracker = dnManager.getSlowPeerTracker(); + assert tracker != null; Review Comment: Use ``assertNotNull`` ########## hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/SlowPeerDisabledTracker.java: ########## @@ -46,11 +46,6 @@ public class SlowPeerDisabledTracker extends SlowPeerTracker { public SlowPeerDisabledTracker(Configuration conf, Timer timer) { super(conf, timer); - final boolean dataNodePeerStatsEnabledVal = - conf.getBoolean(DFSConfigKeys.DFS_DATANODE_PEER_STATS_ENABLED_KEY, - DFSConfigKeys.DFS_DATANODE_PEER_STATS_ENABLED_DEFAULT); - Preconditions.checkArgument(!dataNodePeerStatsEnabledVal, - "SlowPeerDisabledTracker should only be used for disabled slow peer stats."); Review Comment: Nice catch!!! This looks like a prod issue where reconfig won't work itself I believe, you can't reconfig from ``true`` to ``false`` I was able to reproduce the issue by fixing the original test. I think we should fix that test as well ``` diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/TestNameNodeReconfigure.java b/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/TestNameNodeReconfigure.java index 5a0f62a8117e..2bb1c124c90c 100644 - > Fix the logic for reconfigure slow peer enable for Namenode. > ------------------------------------------------------------ > > Key: HDFS-17260 > URL: https://issues.apache.org/jira/browse/HDFS-17260 > Project: Hadoop HDFS > Issue Type: Bug > Reporter: huangzhaobo99 > Priority: Major > Labels: pull-request-available > -- This message was sent by Atlassian Jira (v8.20.10#820010) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org