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 ha...@apache.org on 2015/03/21 22:32:49 UTC

hadoop git commit: MAPREDUCE-5183. In, TaskTracker#reportProgress logging of 0.0-1.0 progress is followed by percent sign. Contributed by Niranjan Singh.

Repository: hadoop
Updated Branches:
  refs/heads/branch-1 3e3623818 -> 552bafde0


MAPREDUCE-5183. In, TaskTracker#reportProgress logging of 0.0-1.0 progress is followed by percent sign. Contributed by Niranjan Singh.


Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo
Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/552bafde
Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/552bafde
Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/552bafde

Branch: refs/heads/branch-1
Commit: 552bafde0f117504b7ef41a1d83a0b593623a7ae
Parents: 3e36238
Author: Harsh J <ha...@cloudera.com>
Authored: Sun Mar 22 03:02:36 2015 +0530
Committer: Harsh J <ha...@cloudera.com>
Committed: Sun Mar 22 03:02:36 2015 +0530

----------------------------------------------------------------------
 CHANGES.txt                                          | 3 +++
 src/mapred/org/apache/hadoop/mapred/TaskTracker.java | 4 ++--
 2 files changed, 5 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/552bafde/CHANGES.txt
----------------------------------------------------------------------
diff --git a/CHANGES.txt b/CHANGES.txt
index b71eb08..6abf1f9 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -66,6 +66,9 @@ Release 1.3.0 - unreleased
 
   BUG FIXES
 
+    MAPREDUCE-5183. In, TaskTracker#reportProgress logging of 0.0-1.0 progress
+    is followed by percent sign (Niranjan Singh via harsh)
+
     MAPREDUCE-5660. Log info about possible thrashing (when using
     memory-based scheduling in Capacity Scheduler) is not printed
     (Adam Kawa via harsh)

http://git-wip-us.apache.org/repos/asf/hadoop/blob/552bafde/src/mapred/org/apache/hadoop/mapred/TaskTracker.java
----------------------------------------------------------------------
diff --git a/src/mapred/org/apache/hadoop/mapred/TaskTracker.java b/src/mapred/org/apache/hadoop/mapred/TaskTracker.java
index 3698eab..18c466c 100644
--- a/src/mapred/org/apache/hadoop/mapred/TaskTracker.java
+++ b/src/mapred/org/apache/hadoop/mapred/TaskTracker.java
@@ -2936,8 +2936,8 @@ public class TaskTracker implements MRConstants, TaskUmbilicalProtocol,
      */
     public synchronized void reportProgress(TaskStatus taskStatus) 
     {
-      LOG.info(task.getTaskID() + " " + taskStatus.getProgress() + 
-          "% " + taskStatus.getStateString());
+      LOG.info(task.getTaskID() + " " + StringUtils.formatPercent(
+          taskStatus.getProgress(), 0) + " " + taskStatus.getStateString());
       // task will report its state as
       // COMMIT_PENDING when it is waiting for commit response and 
       // when it is committing.