Merge -r628214:628215 from trunk to branch 0.16 to fix HADOOP-2811.

git-svn-id: https://svn.apache.org/repos/asf/hadoop/core/branches/branch-0.16@628222 13f79535-47bb-0310-9956-ffa450edef68

CHANGES.txt

@@ -47,6 +47,9 @@ Release 0.16.1 - Unrelease
     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

src/java/org/apache/hadoop/mapred/Counters.java

@@ -519,8 +519,8 @@ public class Counters implements Writable, Iterable<Counters.Group> {
    */
   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;