[
https://issues.apache.org/jira/browse/TEZ-717?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14078784#comment-14078784
]
Jonathan Eagles commented on TEZ-717:
-------------------------------------
Thanks for the review, [~sseth]. I have addressed the issues above.
bq. The DAGAppMasater wait state loop - should this break out on SUCCEEDED as
well ? (unlikely but a super fast DAG). Also, for subsequent enhancements, it
may be possible to just have the DAGAM thread notify once initAndStartAppMaster
completes.
The loop is set up to delay break out until start has been called so as not to
give a dag client handler that will NPE to the client. So to the best of my
understanding, SUCCEEDED will break out as well.
> Client changes to allow local mode DAG submission
> -------------------------------------------------
>
> Key: TEZ-717
> URL: https://issues.apache.org/jira/browse/TEZ-717
> Project: Apache Tez
> Issue Type: Sub-task
> Reporter: Siddharth Seth
> Assignee: Jonathan Eagles
> Priority: Blocker
> Attachments: TEZ-717-v10.patch, TEZ-717-v12.patch, TEZ-717-v13.patch,
> TEZ-717-v14.patch, TEZ-717-v6.patch, TEZ-717-v8.patch, TEZ-717-v9.patch,
> TEZ-717.patch, TEZ-717.patch
>
>
--
This message was sent by Atlassian JIRA
(v6.2#6252)
