[
https://issues.apache.org/jira/browse/TEZ-1303?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14079974#comment-14079974
]
Bikas Saha commented on TEZ-1303:
---------------------------------
Minor. Doesnt seem necessary to have emContext, vmContext,
outputCommitterContext etc. Given that the object is scoped inside the parent
object just context should be enough.
MergedLogicalInput's initialize could also be changed to a constructor.
Looks good.
> 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)
