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

Siddharth Seth commented on TEZ-1057:
-------------------------------------

Pretty much all methods in the Processor are only meant to be invoked by the 
Framework. Essentially, we're removing the Processor interface altogether 
right, primarily for consistency with the Input/Output interfaces ?

Should initialize() in AbstractLogicalProcessor/Input/Output be non-abstract - 
most I/O/P implementations have this as an empty function. Doesn't need to be 
part of this jira though.

Looks like the patch has bits of TEZ-1346 mixed in, so it doesn't apply.

Otherwise looks good.



> Replace interfaces with abstract classes for Processor/Input/Output classes
> ---------------------------------------------------------------------------
>
>                 Key: TEZ-1057
>                 URL: https://issues.apache.org/jira/browse/TEZ-1057
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Bikas Saha
>            Assignee: Bikas Saha
>            Priority: Blocker
>         Attachments: TEZ-1057.1.patch
>
>




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

Reply via email to