Review Board 1.7.22


New logical plan: ImplicitSplitInserter should before DuplicateForEachColumnRewrite

Review Request #167 - Created Dec. 14, 2010 and submitted

Daniel Dai
PIG-1766
Reviewers
pig
xuefu
pig
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)
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