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

Julien Le Dem commented on PIG-3307:
------------------------------------

[~daijy] This is removing parameters that were not used. I have not tested 
performance but I think it could only improve performance.
(see latest patch PIG-3307_2.patch)
                
> Refactor physical operators to remove methods parameters that are always null
> -----------------------------------------------------------------------------
>
>                 Key: PIG-3307
>                 URL: https://issues.apache.org/jira/browse/PIG-3307
>             Project: Pig
>          Issue Type: Improvement
>            Reporter: Julien Le Dem
>            Assignee: Julien Le Dem
>         Attachments: PIG-3307_0.patch, PIG-3307_1.patch, PIG-3307_2.patch
>
>
> The physical operators are sometimes overly complex. I'm trying to cleanup 
> some unnecessary code.
> in particular there is an array of getNext(*T* v) where the value v does not 
> seem to have any importance and is just used to pick the correct method.
> I have started a refactoring for a more readable getNext*T*().

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to