-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1817/
-----------------------------------------------------------
Review request for pig, Daniel Dai and Dmitriy Ryaboy.
Summary
-------
See PIG-2228
This addresses bug PIG-2228.
https://issues.apache.org/jira/browse/PIG-2228
Diffs
-----
trunk/src/org/apache/pig/Algebraic.java 1164722
trunk/src/org/apache/pig/Main.java 1164722
trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/CombinerOptimizer.java
1164722
trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/MapReduceLauncher.java
1164722
trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PhyPlanSetter.java
1164722
trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/plans/EndOfAllInputSetter.java
1164722
trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/plans/PhyPlanVisitor.java
1164722
trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/plans/PlanPrinter.java
1164722
trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POLocalRearrange.java
1164722
trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POPartialAgg.java
PRE-CREATION
trunk/src/org/apache/pig/data/DefaultTuple.java 1164722
trunk/src/org/apache/pig/data/InternalCachedBag.java 1164722
trunk/src/org/apache/pig/data/InternalDistinctBag.java 1164722
trunk/src/org/apache/pig/data/InternalSortedBag.java 1164722
trunk/src/org/apache/pig/data/SelfSpillBag.java PRE-CREATION
trunk/src/org/apache/pig/data/SizeUtil.java PRE-CREATION
trunk/src/org/apache/pig/data/SortedSpillBag.java 1164722
trunk/test/e2e/pig/tests/nightly.conf 1164722
trunk/test/org/apache/pig/test/TestDataBag.java 1164722
trunk/test/org/apache/pig/test/TestPOPartialAgg.java PRE-CREATION
trunk/test/org/apache/pig/test/TestPOPartialAggPlan.java PRE-CREATION
trunk/test/org/apache/pig/test/Util.java 1164722
trunk/test/org/apache/pig/test/utils/GenPhyOp.java 1164722
Diff: https://reviews.apache.org/r/1817/diff
Testing
-------
test-patch
[exec] -1 overall.
[exec]
[exec] +1 @author. The patch does not contain any @author tags.
[exec]
[exec] +1 tests included. The patch appears to include 21 new or
modified tests.
[exec]
[exec] +1 javadoc. The javadoc tool did not generate any warning
messages.
[exec]
[exec] +1 javac. The applied patch does not increase the total number
of javac compiler warnings.
[exec]
[exec] +1 findbugs. The patch does not introduce any new Findbugs
warnings.
[exec]
[exec] -1 release audit. The applied patch generated 461 release
audit warnings (more than the trunk's current 455 warnings).
release audit failures are because of jdiff changes
All unit tests pass, new e2e tests added .
Thanks,
Thejas
