[ 
https://issues.apache.org/jira/browse/HDFS-16528?focusedWorklogId=758914&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-758914
 ]

ASF GitHub Bot logged work on HDFS-16528:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 20/Apr/22 01:38
            Start Date: 20/Apr/22 01:38
    Worklog Time Spent: 10m 
      Work Description: tomscut commented on code in PR #4186:
URL: https://github.com/apache/hadoop/pull/4186#discussion_r853655682


##########
hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/DatanodeManager.java:
##########
@@ -366,6 +365,21 @@ public class DatanodeManager {
         
DFSConfigKeys.DFS_NAMENODE_BLOCKS_PER_POSTPONEDBLOCKS_RESCAN_KEY_DEFAULT);
   }
 
+  /**
+   * Determines whether slow peer tracker should be enabled. If 
dataNodePeerStatsEnabledVal is
+   * true, slow peer tracker is initialized.
+   *
+   * @param conf The configuration to use while initializing slowPeerTracker.
+   * @param timer Timer object for slowPeerTracker.
+   * @param dataNodePeerStatsEnabledVal To determine whether slow peer 
tracking should be enabled.
+   */
+  public void initSlowPeerTracker(Configuration conf, Timer timer,
+      boolean dataNodePeerStatsEnabledVal) {
+    this.dataNodePeerStatsEnabled = dataNodePeerStatsEnabledVal;
+    this.slowPeerTracker = dataNodePeerStatsEnabled ?
+        new SlowPeerTracker(conf, timer) : null;

Review Comment:
   If `this.slowPeerTracker` is set to null directly, may cause NPE. 





Issue Time Tracking
-------------------

    Worklog Id:     (was: 758914)
    Time Spent: 1h 20m  (was: 1h 10m)

> Reconfigure slow peer enable for Namenode
> -----------------------------------------
>
>                 Key: HDFS-16528
>                 URL: https://issues.apache.org/jira/browse/HDFS-16528
>             Project: Hadoop HDFS
>          Issue Type: Task
>            Reporter: Viraj Jasani
>            Assignee: Viraj Jasani
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 1h 20m
>  Remaining Estimate: 0h
>
> HDFS-16396 provides reconfig options for several configs associated with 
> slownodes in Datanode. Similarly, HDFS-16287 and HDFS-16327 have added some 
> slownodes related configs as the reconfig options in Namenode.
> The purpose of this Jira is to add DFS_DATANODE_PEER_STATS_ENABLED_KEY as 
> reconfigurable option for Namenode (similar to how HDFS-16396 has included it 
> for Datanode).



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to