-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43044/
-----------------------------------------------------------
(Updated Feb. 3, 2016, 6:23 a.m.)
Review request for pig, Xianda Ke, liyun zhang, Mohit Sabharwal, and Xuefu
Zhang.
Changes
-------
Fixed some UT failures in UTs other than TestCombiner.
Bugs: PIG-4766
https://issues.apache.org/jira/browse/PIG-4766
Repository: pig-git
Description
-------
PIG-4709 introduced Combiner optimization for Group By. However, the patch did
not handle cases where constant/conditional expressions were used. It also did
not handle limit.
This patch is to address those gaps.
Diffs (updated)
-----
src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/PORelationToExprProject.java
5fb49e2
src/org/apache/pig/backend/hadoop/executionengine/spark/converter/ReduceByConverter.java
d4b521a
src/org/apache/pig/backend/hadoop/executionengine/spark/optimizer/CombinerOptimizer.java
a05d009
src/org/apache/pig/backend/hadoop/executionengine/util/CombinerOptimizerUtil.java
5c0919f
src/org/apache/pig/data/SelfSpillBag.java 4e08b99
test/org/apache/pig/newplan/logical/relational/TestLocationInPhysicalPlan.java
0e45434
test/org/apache/pig/test/TestCombiner.java b2e81ac
Diff: https://reviews.apache.org/r/43044/diff/
Testing
-------
With this patch, all tests in TestCombiner pass.
Thanks,
Pallavi Rao
