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

Cheolsoo Park commented on PIG-4375:
------------------------------------

[~rohini], I haven't tested join + group by. But I tested another job that was 
failing with OOM previously, and it passes now. So +1.

> ObjectCache should use ProcessorContext.getObjectRegistry()
> -----------------------------------------------------------
>
>                 Key: PIG-4375
>                 URL: https://issues.apache.org/jira/browse/PIG-4375
>             Project: Pig
>          Issue Type: Sub-task
>    Affects Versions: 0.14.0
>            Reporter: Rohini Palaniswamy
>            Assignee: Rohini Palaniswamy
>             Fix For: 0.14.1, 0.15.0
>
>         Attachments: PIG-4375-1.patch
>
>
>   We have been instantiating our ObjectRegistryImpl which is wrong and will 
> leak memory. It was a copy from hive code in the beginning and I think lot 
> has changed in Tez after that and we have missed it. We also need to expose 
> the ObjectRegistry to users so that they can cache their own stuff for 
> IndexedLoadFunc implementations.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to