[ https://issues.apache.org/jira/browse/PIG-3417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15762008#comment-15762008 ]
Rohini Palaniswamy commented on PIG-3417: ----------------------------------------- +1. Thanks for the changes. The patch is lot simpler with this. bq. We might not even need to have testSkewJoinWithTuples, since the e2e tests now cover this case. Yes. That should be fine as it is kind of doing exactly what the e2e is doing even for verification. If it was local mode+ mock.Storage combo it would have taken less time, but this one would take more than a minute. Considering that, will skip the unit test while checking in. [~njw45] / [~daijy], Do you want this to go into 0.16.1 as well? > Skewed Join On Tuple Column Kills Job > -------------------------------------- > > Key: PIG-3417 > URL: https://issues.apache.org/jira/browse/PIG-3417 > Project: Pig > Issue Type: Bug > Components: impl > Affects Versions: 0.11.1 > Reporter: Nick White > Assignee: Nandor Kollar > Priority: Critical > Fix For: 0.17.0 > > Attachments: PIG-3417.patch, PIG-3417_2.patch, > TestSkewJoinWithTuples.java > > > I've attached a test case that fails, but should pass. The test case groups > two relations separately, then full-outer joins them on the grouped columns. > The test case passes if "using 'skewed'" is removed. -- This message was sent by Atlassian JIRA (v6.3.4#6332)