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

Pradeep Kamath commented on PIG-1363:
-------------------------------------

+1

> Unnecessary loadFunc instantiations
> -----------------------------------
>
>                 Key: PIG-1363
>                 URL: https://issues.apache.org/jira/browse/PIG-1363
>             Project: Pig
>          Issue Type: Bug
>    Affects Versions: 0.7.0
>            Reporter: Ashutosh Chauhan
>            Assignee: Ashutosh Chauhan
>             Fix For: 0.8.0
>
>         Attachments: pig-1363.patch
>
>
> In MRCompiler loadfuncs are instantiated at multiple locations in different 
> visit methods. This is inconsistent and confusing. LoadFunc should be 
> instantiated at only one place, ideally in LogToPhyTanslation#visit(LOLoad). 
> A getter should be added to POLoad to retrieve this instantiated loadFunc 
> wherever it is needed in later stages of compilation. 

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to