You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tez.apache.org by "Jeff Zhang (JIRA)" <ji...@apache.org> on 2015/04/22 10:46:00 UTC
[jira] [Comment Edited] (TEZ-2261) Should add diagnostics in
DAGAppMaster when recovery error happens
[ https://issues.apache.org/jira/browse/TEZ-2261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14486815#comment-14486815 ]
Jeff Zhang edited comment on TEZ-2261 at 4/22/15 8:45 AM:
----------------------------------------------------------
Upload new patch to fix one dead lock issue in the last patch. [~hitesh] Please help review it.
was (Author: zjffdu):
Upload new patch to fix one dead lock issue in the last patch.
> Should add diagnostics in DAGAppMaster when recovery error happens
> ------------------------------------------------------------------
>
> Key: TEZ-2261
> URL: https://issues.apache.org/jira/browse/TEZ-2261
> Project: Apache Tez
> Issue Type: Bug
> Reporter: Jeff Zhang
> Assignee: Jeff Zhang
> Attachments: TEZ-2261-1.patch, TEZ-2261-2.patch, TEZ-2261-3.patch
>
>
> Should add diagnostics in DAGAppMaster when recovery error happens, otherwise AM is shutdown and the next dag submission will just throw SessionNotRunningException which would confuse users.
> {code}
> if (this.historyEventHandler.hasRecoveryFailed()) {
> LOG.warn("Recovery had a fatal error, shutting down session after" +
> " DAG completion");
> sessionStopped.set(true);
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)