From 91472c553954a08867bce6ef3defa7a96912047f Mon Sep 17 00:00:00 2001 From: Harsh J Date: Wed, 11 Jul 2012 15:07:34 +0000 Subject: [PATCH] Backport of HADOOP-3886 from trunk. svn merge -c 1360222 ../../trunk (harsh) git-svn-id: https://svn.apache.org/repos/asf/hadoop/common/branches/branch-2@1360224 13f79535-47bb-0310-9956-ffa450edef68 --- hadoop-common-project/hadoop-common/CHANGES.txt | 3 +++ .../src/main/java/org/apache/hadoop/util/Progressable.java | 2 +- .../src/main/java/org/apache/hadoop/mapred/Mapper.java | 2 +- .../src/main/java/org/apache/hadoop/mapred/Reporter.java | 2 +- 4 files changed, 6 insertions(+), 3 deletions(-) diff --git a/hadoop-common-project/hadoop-common/CHANGES.txt b/hadoop-common-project/hadoop-common/CHANGES.txt index 6caf1aa951c..9d219ca48fe 100644 --- a/hadoop-common-project/hadoop-common/CHANGES.txt +++ b/hadoop-common-project/hadoop-common/CHANGES.txt @@ -143,6 +143,9 @@ Release 2.0.1-alpha - UNRELEASED HADOOP-8586. Fixup a bunch of SPNEGO misspellings. (eli) + HADOOP-3886. Error in javadoc of Reporter, Mapper and Progressable + (Jingguo Yao via harsh) + BREAKDOWN OF HDFS-3042 SUBTASKS HADOOP-8220. ZKFailoverController doesn't handle failure to become active diff --git a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/Progressable.java b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/Progressable.java index e1f2cb9d957..495ca82b76e 100644 --- a/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/Progressable.java +++ b/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/Progressable.java @@ -26,7 +26,7 @@ * *

Clients and/or applications can use the provided Progressable * to explicitly report progress to the Hadoop framework. This is especially - * important for operations which take an insignificant amount of time since, + * important for operations which take significant amount of time since, * in-lieu of the reported progress, the framework has to assume that an error * has occured and time-out the operation.

*/ diff --git a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/Mapper.java b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/Mapper.java index 2b1362c55c7..eaa6c2b72c4 100644 --- a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/Mapper.java +++ b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/Mapper.java @@ -144,7 +144,7 @@ public interface Mapper extends JobConfigurable, Closeable { * *

Applications can use the {@link Reporter} provided to report progress * or just indicate that they are alive. In scenarios where the application - * takes an insignificant amount of time to process individual key/value + * takes significant amount of time to process individual key/value * pairs, this is crucial since the framework might assume that the task has * timed-out and kill that task. The other way of avoiding this is to set * diff --git a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/Reporter.java b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/Reporter.java index 82ba71eb095..01a50b7f600 100644 --- a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/Reporter.java +++ b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapred/Reporter.java @@ -29,7 +29,7 @@ * *

{@link Mapper} and {@link Reducer} can use the Reporter * provided to report progress or just indicate that they are alive. In - * scenarios where the application takes an insignificant amount of time to + * scenarios where the application takes significant amount of time to * process individual key/value pairs, this is crucial since the framework * might assume that the task has timed-out and kill that task. *