-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25617/
-----------------------------------------------------------
(Updated Sept. 14, 2014, 5:56 a.m.)
Review request for pig, Cheolsoo Park and Daniel Dai.
Changes
-------
Reuse the same TezAccumulativeTupleBuffer for all input keys.
Bugs: PIG-4104
https://issues.apache.org/jira/browse/PIG-4104
Repository: pig
Description
-------
Use a separate TezAccumulativeTupleBuffer that iterates through the inputs and
returns tuples in batches instead of making a full copy.
Diffs (updated)
-----
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/PigConfiguration.java
1624398
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POForEach.java
1624398
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POPackage.java
1624398
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/tez/POShuffleTezLoad.java
1624398
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/util/AccumulatorOptimizerUtil.java
1624398
Diff: https://reviews.apache.org/r/25617/diff/
Testing
-------
Ran TestAccumulator in unit test and Accumulator, SecondarySort test groups in
e2e and they all passed. Will run the full suite before committing.
Thanks,
Rohini Palaniswamy
