You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by iw...@apache.org on 2022/03/20 01:00:04 UTC
[hadoop] branch branch-3.2.3 updated: HDFS-14630. Configuration.getTimeDurationHelper() should not log time unit warning in info log. Contributed by hemanthboyina.
This is an automated email from the ASF dual-hosted git repository.
iwasakims pushed a commit to branch branch-3.2.3
in repository https://gitbox.apache.org/repos/asf/hadoop.git
The following commit(s) were added to refs/heads/branch-3.2.3 by this push:
new abe5358 HDFS-14630. Configuration.getTimeDurationHelper() should not log time unit warning in info log. Contributed by hemanthboyina.
abe5358 is described below
commit abe5358143720085498613d399be3bbf01e0f131
Author: Surendra Singh Lilhore <su...@apache.org>
AuthorDate: Tue Sep 3 12:31:23 2019 +0530
HDFS-14630. Configuration.getTimeDurationHelper() should not log time unit warning in info log. Contributed by hemanthboyina.
(cherry picked from commit 5ff76cb8bc69d68ba7c9487d00b1dc753d616bb2)
---
.../java/org/apache/hadoop/conf/Configuration.java | 2 --
.../src/main/resources/hdfs-default.xml | 23 +++++++++++++++-------
2 files changed, 16 insertions(+), 9 deletions(-)
diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/conf/Configuration.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/conf/Configuration.java
old mode 100644
new mode 100755
index 7301af6..215627f
--- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/conf/Configuration.java
+++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/conf/Configuration.java
@@ -1856,8 +1856,6 @@ public class Configuration implements Iterable<Map.Entry<String,String>>,
vStr = StringUtils.toLowerCase(vStr);
ParsedTimeDuration vUnit = ParsedTimeDuration.unitFor(vStr);
if (null == vUnit) {
- logDeprecation("No unit for " + name + "(" + vStr + ") assuming " +
- defaultUnit);
vUnit = ParsedTimeDuration.unitFor(defaultUnit);
} else {
vStr = vStr.substring(0, vStr.lastIndexOf(vUnit.suffix()));
diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/main/resources/hdfs-default.xml b/hadoop-hdfs-project/hadoop-hdfs/src/main/resources/hdfs-default.xml
old mode 100644
new mode 100755
index 0d31b28..5ac2bff
--- a/hadoop-hdfs-project/hadoop-hdfs/src/main/resources/hdfs-default.xml
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/main/resources/hdfs-default.xml
@@ -745,7 +745,8 @@
<value>0</value>
<description>
Delay for first block report in seconds. Support multiple time unit
- suffix(case insensitive), as described in dfs.heartbeat.interval.
+ suffix(case insensitive), as described in dfs.heartbeat.interval.If
+ no time unit is specified then seconds is assumed
</description>
</property>
@@ -790,7 +791,8 @@
<description>Interval in seconds for Datanode to scan data directories and
reconcile the difference between blocks in memory and on the disk.
Support multiple time unit suffix(case insensitive), as described
- in dfs.heartbeat.interval.
+ in dfs.heartbeat.interval.If no time unit is specified then seconds
+ is assumed.
</description>
</property>
@@ -831,6 +833,7 @@
ms(millis), s(sec), m(min), h(hour), d(day)
to specify the time (such as 2s, 2m, 1h, etc.).
Or provide complete number in seconds (such as 30 for 30 seconds).
+ If no time unit is specified then seconds is assumed.
</description>
</property>
@@ -1039,6 +1042,7 @@
<description>Namenode periodicity in seconds to check if
decommission or maintenance is complete. Support multiple time unit
suffix(case insensitive), as described in dfs.heartbeat.interval.
+ If no time unit is specified then seconds is assumed.
</description>
</property>
@@ -1187,7 +1191,8 @@
<description>
The number of seconds between two periodic checkpoints.
Support multiple time unit suffix(case insensitive), as described
- in dfs.heartbeat.interval.
+ in dfs.heartbeat.interval.If no time unit is specified then seconds
+ is assumed.
</description>
</property>
@@ -1206,7 +1211,8 @@
<description>The SecondaryNameNode and CheckpointNode will poll the NameNode
every 'dfs.namenode.checkpoint.check.period' seconds to query the number
of uncheckpointed transactions. Support multiple time unit suffix(case insensitive),
- as described in dfs.heartbeat.interval.
+ as described in dfs.heartbeat.interval.If no time unit is specified then
+ seconds is assumed.
</description>
</property>
@@ -1594,7 +1600,8 @@
The notification is sent by a datanode when it is being shutdown
using the shutdownDatanode admin command with the upgrade option.
Support multiple time unit suffix(case insensitive), as described
- in dfs.heartbeat.interval.
+ in dfs.heartbeat.interval.If no time unit is specified then seconds
+ is assumed.
</description>
</property>
@@ -1663,7 +1670,8 @@
often the logs are rolled. Note that failover triggers a log roll
so the StandbyNode will be up to date before it becomes active.
Support multiple time unit suffix(case insensitive), as described
- in dfs.heartbeat.interval.
+ in dfs.heartbeat.interval.If no time unit is specified then seconds
+ is assumed.
</description>
</property>
@@ -3445,7 +3453,8 @@
datanode is not yet registered with the namenode. This wait time
reduces initial request failures after datanode restart.
Support multiple time unit suffix(case insensitive), as described
- in dfs.heartbeat.interval.
+ in dfs.heartbeat.interval.If no time unit is specified then seconds
+ is assumed.
</description>
</property>
---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org