HDFS-17233. The conf dfs.datanode.lifeline.interval.seconds is not co…

…nsidering time unit seconds (#6286). Contributed by Palakur Eshwitha Sai. Reviewed-by: huhaiyang <[email protected]> Signed-off-by: Ayush Saxena <[email protected]>
apache · Dec 2, 2023 · d0b460f · d0b460f
1 parent e76477e
commit d0b460f
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/...hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/DNConf.java b/...hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/DNConf.java
@@ -220,10 +220,10 @@ public DNConf(final Configurable dn) {
         DFS_HEARTBEAT_INTERVAL_DEFAULT, TimeUnit.SECONDS,
         TimeUnit.MILLISECONDS);
     long confLifelineIntervalMs =
-        getConf().getLong(DFS_DATANODE_LIFELINE_INTERVAL_SECONDS_KEY,
-        3 * getConf().getTimeDuration(DFS_HEARTBEAT_INTERVAL_KEY,
-        DFS_HEARTBEAT_INTERVAL_DEFAULT, TimeUnit.SECONDS,
-            TimeUnit.MILLISECONDS));
+        getConf().getTimeDuration(DFS_DATANODE_LIFELINE_INTERVAL_SECONDS_KEY,
+            3 * getConf().getTimeDuration(DFS_HEARTBEAT_INTERVAL_KEY,
+                DFS_HEARTBEAT_INTERVAL_DEFAULT, TimeUnit.SECONDS),
+            TimeUnit.SECONDS, TimeUnit.MILLISECONDS);
     if (confLifelineIntervalMs <= heartBeatInterval) {
       confLifelineIntervalMs = 3 * heartBeatInterval;
       DataNode.LOG.warn(