[ 
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)

Reply via email to