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

Siddharth Seth commented on TEZ-717:
------------------------------------

bq. 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.
My bad. Misread the checks.

One little issue with the latest patch. 
{code}
Path logDir = new Path(userDir, "logs");
Path localDir = new Path(userDir, "logs");
{code}
Both pointing to the "logs" sub-dir.
The rest looks good. Thanks [~jeagles]. Will make this minor fix while 
committing.

> 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)

Reply via email to