[
https://issues.apache.org/jira/browse/TEZ-1303?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Siddharth Seth updated TEZ-1303:
--------------------------------
Attachment: TEZ-1303.1.wip.txt
Work in progress patch which changes Input constructions.
LogicalInputs now require a constructor - "LogicalInput(TezInputContext, int
numPhysicalInputs)", and setNumPhysicalInputs / initialize(TezInputContext) are
removed.
This works well with AbstractLogicalInput exposing a no-argument initialize()
method - which is now the main interface.
[~bikassaha], [~hitesh] - any thoughts on this ?
One concern I have is that the num physical inputs may not be known at
initialization time (in the future).
> 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
> Priority: Blocker
> Attachments: TEZ-1303.1.wip.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)
