[ https://issues.apache.org/jira/browse/PIG-5215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16024321#comment-16024321 ]
liyunzhang_intel commented on PIG-5215: --------------------------------------- [~szita]: update PIG-5215.4.patch, compilation pass while i am testing it in my local jenkins. fix all comments on [review board|https://reviews.apache.org/r/57317] except 1: {noformat} Testing distinct + orderby + limit serves the same purpose as orderby + limit test. Can you remove orderby from this test? If distinct + limit differs everytime even with spark and a different verify_pig_script runs just ignore the test for now adding a TODO to test num 4. {noformat} I think we can work in parallel( I continue to test the patch while you start to review) as there is not much time before final release. please help review the patch and fix remaining issue as you and [~nkollar] have more knowledge on Limit_4 e2e test. > Merge changes from review board to spark branch > ----------------------------------------------- > > Key: PIG-5215 > URL: https://issues.apache.org/jira/browse/PIG-5215 > Project: Pig > Issue Type: Bug > Components: spark > Reporter: liyunzhang_intel > Fix For: spark-branch > > Attachments: PIG-5215.1.patch, PIG-5215.3.patch, PIG-5215.patch > > > in [review board|https://reviews.apache.org/r/57317/], there are comments > from community. After the review board is close, merge these changes to spark > branch -- This message was sent by Atlassian JIRA (v6.3.15#6346)