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

Bikas Saha commented on TEZ-1304:
---------------------------------

Is direct method invocation fine on DagAppMaster? If we do that then 
DAGClientServer code is completely untested when the tests start using local 
mode. IMO, just abstracting YarnClient for now would be a smaller/simpler 
change that gives us most test coverage. The less code we change now, the less 
surprises for us when we actually try local mode because we wont be working 
around code duplication or missing compatibility. If performance becomes an 
issue we could make improvements to do direct method invocations. Something to 
think about.

> Create execution underlying framework abstraction layer
> -------------------------------------------------------
>
>                 Key: TEZ-1304
>                 URL: https://issues.apache.org/jira/browse/TEZ-1304
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Jonathan Eagles
>            Assignee: Jonathan Eagles
>         Attachments: TEZ-1304-v1.patch, TEZ-1304-v2.patch
>
>
> Continuing in the direction of hiding the internals of the execution 
> framework, this patch aims at abstracting it away from the tez client 
> implementation. This will further ease the addition of other execution 
> frameworks such as Tez Local Mode.



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

Reply via email to