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

Rajesh Balamohan commented on TEZ-2591:
---------------------------------------

Earlier it was handling both cases (i.e routing model & onDemand).  Now 
EdgeManagerPluginBase becomes the base which is extended by EdgeManagerPlugin 
and EdgeManagerPluginOnDemand.  However, almost all impls except 
OneToOneEdgeManager are based on EdgeManagerPluginOnDemand.  Would this be a 
problem for other jiras? (e.g TEZ-2255). For example, do we need to have 
separate set of impls for EdgeManagerPlugin based approach?

> Remove unneeded methods from EdgeManagerPludinOnDemand
> ------------------------------------------------------
>
>                 Key: TEZ-2591
>                 URL: https://issues.apache.org/jira/browse/TEZ-2591
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Bikas Saha
>            Assignee: Bikas Saha
>         Attachments: TEZ-2591.1.patch, TEZ-2591.2.patch, TEZ-2591.3.patch
>
>
> EdgeManagerPluginOnDemand inherits from EdgeManager (legacy) due to some 
> common methods. Removing this dependency would be cleaner for the code.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to