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

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

That could go inside here. FramewordClient.createYarnClient() could check 
inside it whether we need a real cluster client or a local fake client. We 
could rename to FrameworkClient.createFrameworkClient().
{code}   // for testing
-  protected YarnClient createYarnClient() {
-    return YarnClient.createYarnClient();
+  protected FrameworkClient createYarnClient() {
+    return FrameworkClient.createYarnClient();
   }{code}

> 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
>            Priority: Blocker
>         Attachments: TEZ-1304-v1.patch, TEZ-1304-v2.patch, TEZ-1304-v3.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