pig-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From cheol...@apache.org
Subject svn commit: r1604050 - in /pig/trunk: CHANGES.txt src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POPartialAgg.java
Date Thu, 19 Jun 2014 23:58:55 GMT
Author: cheolsoo
Date: Thu Jun 19 23:58:55 2014
New Revision: 1604050

URL: http://svn.apache.org/r1604050
Log:
PIG-4001: POPartialAgg aggregates too aggressively when multiple values aggregated (tmwoodruff
via cheolsoo)

Modified:
    pig/trunk/CHANGES.txt
    pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POPartialAgg.java

Modified: pig/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/trunk/CHANGES.txt?rev=1604050&r1=1604049&r2=1604050&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Thu Jun 19 23:58:55 2014
@@ -36,6 +36,8 @@ OPTIMIZATIONS
  
 BUG FIXES
 
+PIG-4001: POPartialAgg aggregates too aggressively when multiple values aggregated (tmwoodruff
via cheolsoo)
+
 PIG-4027: Always check for latest Tez snapshot dependencies (lbendig via cheolsoo)
 
 PIG-4020: Fix tez e2e tests MapPartialAgg_[2-4], StreamingPerformance_[6-7] (daijy)

Modified: pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POPartialAgg.java
URL: http://svn.apache.org/viewvc/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POPartialAgg.java?rev=1604050&r1=1604049&r2=1604050&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POPartialAgg.java
(original)
+++ pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POPartialAgg.java
Thu Jun 19 23:58:55 2014
@@ -353,7 +353,7 @@ public class POPartialAgg extends Physic
         Result res = getOutput(key, valueTuple);
         rawInputMap.remove(key);
         addKeyValToMap(processedInputMap, key, getAggResultTuple(res.result));
-        numRecsInProcessedMap += valueTuple.size() - 1;
+        numRecsInProcessedMap++;
     }
 
     /**



Mime
View raw message