svn commit: r628222 - in /hadoop/core/branches/branch-0.16: CHANGES.txt src/java/org/apache/hadoop/mapred/Counters.java

[omalley]

Author: omalley
Date: Fri Feb 15 17:06:40 2008
New Revision: 628222

URL: http://svn.apache.org/viewvc?rev=628222&view=rev
Log:
Merge -r628214:628215 from trunk to branch 0.16 to fix HADOOP-2811.

Modified:
    hadoop/core/branches/branch-0.16/CHANGES.txt
    
hadoop/core/branches/branch-0.16/src/java/org/apache/hadoop/mapred/Counters.java

Modified: hadoop/core/branches/branch-0.16/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/core/branches/branch-0.16/CHANGES.txt?rev=628222&r1=628221&r2=628222&view=diff
==============================================================================
--- hadoop/core/branches/branch-0.16/CHANGES.txt (original)
+++ hadoop/core/branches/branch-0.16/CHANGES.txt Fri Feb 15 17:06:40 2008
@@ -47,6 +47,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/branches/branch-0.16/src/java/org/apache/hadoop/mapred/Counters.java
URL: 
http://svn.apache.org/viewvc/hadoop/core/branches/branch-0.16/src/java/org/apache/hadoop/mapred/Counters.java?rev=628222&r1=628221&r2=628222&view=diff
==============================================================================
--- 
hadoop/core/branches/branch-0.16/src/java/org/apache/hadoop/mapred/Counters.java
 (original)
+++ 
hadoop/core/branches/branch-0.16/src/java/org/apache/hadoop/mapred/Counters.java
 Fri Feb 15 17:06:40 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;