You are viewing a plain text version of this content. The canonical link for it is here.
Posted to mapreduce-commits@hadoop.apache.org by sh...@apache.org on 2009/07/27 10:34:41 UTC

svn commit: r798078 - in /hadoop/mapreduce/trunk: CHANGES.txt src/c++/pipes/impl/HadoopPipes.cc

Author: sharad
Date: Mon Jul 27 08:34:41 2009
New Revision: 798078

URL: http://svn.apache.org/viewvc?rev=798078&view=rev
Log:
MAPREDUCE-383. Fix a bug in Pipes combiner due to bytes count not  getting reset after the spill. Contributed by Christian Kunz.

Modified:
    hadoop/mapreduce/trunk/CHANGES.txt
    hadoop/mapreduce/trunk/src/c++/pipes/impl/HadoopPipes.cc

Modified: hadoop/mapreduce/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/CHANGES.txt?rev=798078&r1=798077&r2=798078&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/CHANGES.txt (original)
+++ hadoop/mapreduce/trunk/CHANGES.txt Mon Jul 27 08:34:41 2009
@@ -262,3 +262,6 @@
 
     MAPREDUCE-765. Eliminate the deprecated warnings introduced by H-5438.
     (He Yongqiang via szetszwo)
+
+    MAPREDUCE-383. Fix a bug in Pipes combiner due to bytes count not 
+    getting reset after the spill. (Christian Kunz via sharad)

Modified: hadoop/mapreduce/trunk/src/c++/pipes/impl/HadoopPipes.cc
URL: http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/src/c%2B%2B/pipes/impl/HadoopPipes.cc?rev=798078&r1=798077&r2=798078&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/src/c++/pipes/impl/HadoopPipes.cc (original)
+++ hadoop/mapreduce/trunk/src/c++/pipes/impl/HadoopPipes.cc Mon Jul 27 08:34:41 2009
@@ -597,6 +597,7 @@
         combiner->reduce(context);
       }
       data.clear();
+      numBytes = 0;
     }
   };