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

Pradeep Kamath commented on PIG-559:
------------------------------------

+1 - the only comment is whether the following check is required :
{code}
if (input == null || input.size() == 0)
            return null;
{code}

In POUserFunc if the input is not directly attached to POUserFunc, it create a 
new Tuple and fills it with the getNext() result of its input. To my knowledge 
currently the POUserFunc will not be the root of Foreach inner plan and hence 
input should not be directly attached. So could these checks be avoided? 

> ARITY and SIZE function produce wrong value for tuples
> ------------------------------------------------------
>
>                 Key: PIG-559
>                 URL: https://issues.apache.org/jira/browse/PIG-559
>             Project: Pig
>          Issue Type: Bug
>            Reporter: Olga Natkovich
>            Assignee: Olga Natkovich
>         Attachments: PIG-559.patch
>
>
> As Santhosh pointed out the problem is that instead of taking the first 
> element of the input tuple and applying size to it, the code just applices 
> the size to the input tuple which is always 1.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to