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

Bikas Saha commented on TEZ-857:
--------------------------------

The type of the object being casted is already LogicalInput in this case. The 
rest of the patch for this file already has unchecked casts.

For the naming, LogicalInputInternal may be a concise name?

> Split Input/Output interfaces into Framework / User components
> --------------------------------------------------------------
>
>                 Key: TEZ-857
>                 URL: https://issues.apache.org/jira/browse/TEZ-857
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Siddharth Seth
>            Assignee: Siddharth Seth
>         Attachments: TEZ-857.1.txt
>
>
> Inputs / Outputs have several methods which are not meant for user 
> interaction - initialize(Tez*Context), close(), TEZ-844 is adding another.
> There has been confusion in the past on whether the framework will call 
> close, or whether it's the user's responsibility.
> The framework specific methods and the Processor usable methods can be split 
> into a separate interface. Input/Output writers would need to implement both, 
> Processor writers would only see the Input part of the interface.
> TEZ-782, TEZ-827 introduced some requirements on Inputs which are not 
> enforced at compile time (must request mem). These could potentially be added 
> to the framework part of the interface to force Inputs/Outputs to be aware of 
> them at compile time, while not polluting the Processor interface.



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

Reply via email to