[ 
https://issues.apache.org/jira/browse/PIG-1963?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13017339#comment-13017339
 ] 

Daniel Dai commented on PIG-1963:
---------------------------------

This prevent other nested relational operator, right? From the way 
checkUDFInput handles PORelationToExprProject, it seems the original code 
intend to prevent other relational operator (except foreach) as well, but miss 
the case POProject following PORelationToExprProject. +1 for the patch if this 
is the case.

> in nested foreach, accumutive udf taking input from order-by does not get 
> results in order
> ------------------------------------------------------------------------------------------
>
>                 Key: PIG-1963
>                 URL: https://issues.apache.org/jira/browse/PIG-1963
>             Project: Pig
>          Issue Type: Bug
>    Affects Versions: 0.8.0, 0.9.0
>            Reporter: Thejas M Nair
>             Fix For: 0.8.0, 0.9.0
>
>         Attachments: MYCONCATBAG.java, PIG-1963.1.patch
>
>
> This happens only when secondary sort is not being used for the order-by. 
> For example -
> {code}
> a1 = load 'fruits.txt' as (f1:int,f2);
> a2 = load 'fruits.txt' as (f1:int,f2);
> b = cogroup a1 by f1, a2 by f1;
> d = foreach b {
>    sort1 = order a1 by f2;
>    sort2 = order a2 by f2; -- secondary sort not getting used here, 
> MYCONCATBAG gets results in wrong order
>    generate group, MYCONCATBAG(sort1.f1), MYCONCATBAG(sort2.f2);
> }
> -- explain d;
> dump d;
> {code}

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to