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

Jeff Zhang commented on TEZ-387:
--------------------------------

[~bikassaha] Thanks for your review and comment.  It looks like I could get the 
dispatcher from DagAppMaster, so no need to pass the dispatcher through 
constructor.  I update the patch using this way. 

> Move DAGClientHandler into its own class
> ----------------------------------------
>
>                 Key: TEZ-387
>                 URL: https://issues.apache.org/jira/browse/TEZ-387
>             Project: Apache Tez
>          Issue Type: Bug
>    Affects Versions: 0.4.0
>            Reporter: Bikas Saha
>            Assignee: Jeff Zhang
>              Labels: newbie
>         Attachments: TEZ-387-2.patch, TEZ-387.patch, TEZ-387.patch, 
> TEZ-387_1.patch, TEZ-387_3.patch, Tez-387-2.patch, Tez-387.patch
>
>
> DAGClientHandler code lies inside the DAGAppMaster. Moving it into its own 
> class will help unit testing it.



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

Reply via email to