-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/167/
-----------------------------------------------------------
Review request for pig and Xuefu Zhang.
Summary
-------
The following script produce wrong result:
A = load '1.txt' AS (a0:int, a1:int);
B = load '2.txt' AS (b0:int, b1:chararray);
C = join A by a0, B by b0;
D = foreach B generate b0 as d0, b1 as d1;
E = join C by a1, D by d0;
F = foreach E generate b1, d1;
dump F;
1.txt:
1 2
1 3
2 4
2 5
2.txt:
1 one
2 two
Expected:
(one,two)
We get:
(one,one)
This addresses bug PIG-1766.
https://issues.apache.org/jira/browse/PIG-1766
Diffs
-----
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/newplan/logical/optimizer/LogicalPlanOptimizer.java
1048916
http://svn.apache.org/repos/asf/pig/trunk/test/org/apache/pig/test/TestEvalPipeline2.java
1048916
Diff: https://reviews.apache.org/r/167/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 3 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 does not increase the
total number of release audit warnings.
Unit test:
all pass
end-to-end test:
all pass
Thanks,
Daniel
