[jira] [Commented] (TEZ-1844) Shouldn't invoke system.exit in local mode when AM is failed to start

[Jeff Zhang (JIRA)]

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

Jeff Zhang commented on TEZ-1844:
---------------------------------

Thanks [~jeagles], will commit it to master shortly.

 

> Shouldn't invoke system.exit in local mode when AM is failed to start
> ---------------------------------------------------------------------
>
>                 Key: TEZ-1844
>                 URL: https://issues.apache.org/jira/browse/TEZ-1844
>             Project: Apache Tez
>          Issue Type: Bug
>            Reporter: Jeff Zhang
>            Assignee: Jeff Zhang
>         Attachments: TEZ-1844-2.patch, TEZ-1844-3.patch, TEZ-1844.patch
>
>
> In local mode, TezClient and Tez-AM run in the same JVM, we don't need to 
> call system.exit when AM is failed to start, throw exception should be more 
> proper. 
> {code}
>           dagAppMaster = createDAGAppMaster(applicationAttemptId, cId, 
> currentHost, nmPort, nmHttpPort,
>                   new SystemClock(),
>                   appSubmitTime, isSession, userDir.toUri().getPath());
>           clientHandler = new DAGClientHandler(dagAppMaster);
>           DAGAppMaster.initAndStartAppMaster(dagAppMaster, 
> currentUser.getShortUserName());
>           } catch (Throwable t) {
>           LOG.fatal("Error starting DAGAppMaster", t);
>           System.exit(1);
>         }
> {code}



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