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

Siddharth Seth commented on TEZ-1303:
-------------------------------------

Will make the changes and upload another patch. Thanks for the quick review.

I'd avoided the MergedLogicalInput change since that would mean users having to 
implement a constructor - but that's the case for everything else anyway, so 
will make the change.

> Change I/P/O/etc construction to make use of contexts
> -----------------------------------------------------
>
>                 Key: TEZ-1303
>                 URL: https://issues.apache.org/jira/browse/TEZ-1303
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Siddharth Seth
>            Assignee: Siddharth Seth
>            Priority: Blocker
>         Attachments: TEZ-1303.1.wip.txt, TEZ-1303.2.txt
>
>
> Instead of having an empty constructor and setting the context via an 
> initialize method - thus allowing the context to be accessed via a final 
> field.
> Using initialize causes potential issues with visibility since object 
> creation, initialize and access can be on completely different threads.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to