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

Jason Lowe commented on TEZ-3852:
---------------------------------

Thanks for the patch!  Could you elaborate a bit in the JIRA description what's 
going on with this change?

In the isSuperSet method I think we can remove the {{srcVal != null}} check 
since the oVal.equals call should already handle null arguments, and removing 
that lets us eliminate the duplication of debug log statements.

Otherwise patch looks good to me.




> Optimize ContainerContext.isSuperSet to speed container reuse decisions
> -----------------------------------------------------------------------
>
>                 Key: TEZ-3852
>                 URL: https://issues.apache.org/jira/browse/TEZ-3852
>             Project: Apache Tez
>          Issue Type: Improvement
>            Reporter: Jonathan Eagles
>            Assignee: Jonathan Eagles
>         Attachments: TEZ-3852.001.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to