MAPREDUCE-6255. Fix JobCounter's format to use grouping separator. Contributed 
by Ryu Kobayashi.


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

Branch: refs/heads/HDFS-EC
Commit: a4f8c978d4fb4e7ff67c4932a63c50ac88b7af74
Parents: ac5497c
Author: Tsuyoshi Ozawa <oz...@apache.org>
Authored: Fri Feb 13 16:09:54 2015 +0900
Committer: Zhe Zhang <z...@apache.org>
Committed: Mon Feb 16 10:29:49 2015 -0800

----------------------------------------------------------------------
 hadoop-mapreduce-project/CHANGES.txt                           | 3 +++
 .../apache/hadoop/mapreduce/v2/app/webapp/CountersBlock.java   | 6 +++---
 2 files changed, 6 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hadoop/blob/a4f8c978/hadoop-mapreduce-project/CHANGES.txt
----------------------------------------------------------------------
diff --git a/hadoop-mapreduce-project/CHANGES.txt 
b/hadoop-mapreduce-project/CHANGES.txt
index 657b441..38982e4 100644
--- a/hadoop-mapreduce-project/CHANGES.txt
+++ b/hadoop-mapreduce-project/CHANGES.txt
@@ -288,6 +288,9 @@ Release 2.7.0 - UNRELEASED
     MAPREDUCE-4431. mapred command should print the reason on killing already 
     completed jobs. (devaraj via ozawa)
 
+    MAPREDUCE-6255. Fix JobCounter's format to use grouping separator.
+    (Ryu Kobayashi via ozawa)
+
   OPTIMIZATIONS
 
     MAPREDUCE-6169. MergeQueue should release reference to the current item 

http://git-wip-us.apache.org/repos/asf/hadoop/blob/a4f8c978/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/v2/app/webapp/CountersBlock.java
----------------------------------------------------------------------
diff --git 
a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/v2/app/webapp/CountersBlock.java
 
b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/v2/app/webapp/CountersBlock.java
index 1b20c5d..568658e 100644
--- 
a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/v2/app/webapp/CountersBlock.java
+++ 
b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/v2/app/webapp/CountersBlock.java
@@ -139,10 +139,10 @@ public class CountersBlock extends HtmlBlock {
           Counter mc = mg == null ? null : mg.findCounter(counter.getName());
           Counter rc = rg == null ? null : rg.findCounter(counter.getName());
           groupRow.
-            td(mc == null ? "0" : String.valueOf(mc.getValue())).
-            td(rc == null ? "0" : String.valueOf(rc.getValue()));
+            td(mc == null ? "0" : String.format("%,d", mc.getValue())).
+            td(rc == null ? "0" : String.format("%,d", rc.getValue()));
         }
-        groupRow.td(String.valueOf(counter.getValue()))._();
+        groupRow.td(String.format("%,d", counter.getValue()))._();
       }
       group._()._()._()._();
     }

Reply via email to