[
https://issues.apache.org/jira/browse/TEZ-2004?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14659692#comment-14659692
]
Jeff Zhang commented on TEZ-2004:
---------------------------------
Comments:
* Is ContainerOp necessary ? It seems ContainerLauncherOperationBase can be
used instead, just need to move OPType into ContainerLauncherOperationBase,
how about rename it as ContainerOperationBase ?
* TaskCommunicator.java Rename unregisterRunningTaskAttempt to
registerTaskAttemptEnd (make it more consistent)
* Rename ContainerLauncherImpl to TezContainerLauncherImpl ? Make all the
default implementation with prefix Tez ?
* Typo DagTypeConverters.convertServicePluginDescriptoToProto -->
DagTypeConverters.convertServicePluginDescriptorToProto (miss "r")
* Need to verify the DAG's defaultExecutionContext and Vertex's
ExecutionContext exist in the TezClient.servicePluginsDescriptor when
submitting the dag
* Need to verify VertexExecutionContext's executeInAm & executeInContainers is
supported in TezClient's ServicePluginsDescriptor
* Seems currently there's only programmatic way to specify TaskScheduler,
ContainerLauncher, TaskCommunicator through TezClient's
ServicePluginsDescriptor, is it expected ? That would mean if some third party
want to introduce new external service to hive, not only they need to implement
new TaskScheduler, ContainerLauncher, TaskCommunicator but also need to change
hive code and rebuild.
> Define basic interface for pluggable ContainerLaunchers
> -------------------------------------------------------
>
> Key: TEZ-2004
> URL: https://issues.apache.org/jira/browse/TEZ-2004
> Project: Apache Tez
> Issue Type: Sub-task
> Reporter: Siddharth Seth
> Assignee: Siddharth Seth
> Fix For: TEZ-2003
>
> Attachments: TEZ-2004.1.txt
>
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
