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

Jeff Zhang commented on TEZ-2708:
---------------------------------

* DAGAppMaster#createContainerLauncherRouter --> 
DAGAppMaster#createContainerLauncherManager
* TaskCommunicatorContextImpl#taskAttemptListener --> 
TaskCommunicatorContextImpl#taskCommunicatorManager ?
* TaskSchedulerContextImpl#tseh --> TaskSchedulerContextImpl#tsm (or 
taskSchedulerManager)
* MockDAGAppMaster#taListener --> MockDAGAppMaster#taskSchedulerManager

API related question
* Are you going to use TaskCommunicator to replace TaskAttemptListener in the 
future? If no, these related renames could be done in the future (like 
DAGAppMaster#createTaskAttemptListener) 


> renames for tez-2003 changes
> ----------------------------
>
>                 Key: TEZ-2708
>                 URL: https://issues.apache.org/jira/browse/TEZ-2708
>             Project: Apache Tez
>          Issue Type: Sub-task
>    Affects Versions: TEZ-2003
>            Reporter: Siddharth Seth
>            Assignee: Siddharth Seth
>         Attachments: TEZ-2708.1.txt, TEZ-2708.2.txt, TEZ-2708.3.txt, 
> TEZ-2708.3_fulldiff.txt
>
>
> This jira is to track some class renames which are required. TBD just before 
> merging or right after the merge.
> -  ContainerLauncherImpl to TezContainerLauncherImpl ? Make all the default 
> implementation with prefix Tez.
> - TaskAttemptListenerImpTezDag to TaskCommunicatorManager
> - Likewise for tests.
> - Remove TezTaskRunner
> - Rename TaskSchedulerEventHandler



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

Reply via email to