svn commit: r628215 - in /hadoop/core/trunk: CHANGES.txt src/java/org/apache/hadoop/mapred/Counters.java

[omalley]

Author: omalley
Date: Fri Feb 15 16:29:24 2008
New Revision: 628215

URL: http://svn.apache.org/viewvc?rev=628215&view=rev
Log:
HADOOP-2811. Dump of counters in job history does not add comma between
groups. Contributed by Runping Qi.

Modified:
    hadoop/core/trunk/CHANGES.txt
    hadoop/core/trunk/src/java/org/apache/hadoop/mapred/Counters.java

Modified: hadoop/core/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/core/trunk/CHANGES.txt?rev=628215&r1=628214&r2=628215&view=diff
==============================================================================
--- hadoop/core/trunk/CHANGES.txt (original)
+++ hadoop/core/trunk/CHANGES.txt Fri Feb 15 16:29:24 2008
@@ -94,6 +94,9 @@
     HADOOP-2814. Fix for NPE in datanode in unit test TestDataTransferProtocol.
     (Raghu Angadi via dhruba)
 
+    HADOOP-2811. Dump of counters in job history does not add comma between
+    groups. (runping via omalley)
+
 Release 0.16.0 - 2008-02-07
 
   INCOMPATIBLE CHANGES

Modified: hadoop/core/trunk/src/java/org/apache/hadoop/mapred/Counters.java
URL: 
http://svn.apache.org/viewvc/hadoop/core/trunk/src/java/org/apache/hadoop/mapred/Counters.java?rev=628215&r1=628214&r2=628215&view=diff
==============================================================================
--- hadoop/core/trunk/src/java/org/apache/hadoop/mapred/Counters.java (original)
+++ hadoop/core/trunk/src/java/org/apache/hadoop/mapred/Counters.java Fri Feb 
15 16:29:24 2008
@@ -519,8 +519,8 @@
    */
   public synchronized String makeCompactString() {
     StringBuffer buffer = new StringBuffer();
-    for(Group group: this){
-      boolean first = true;
+    boolean first = true;
+    for(Group group: this){   
       for(Counter counter: group) {
         if (first) {
           first = false;