[TEZ-2261] Should add diagnostics in DAGAppMaster when recovery error happens

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.7.0
• Target Version/s: 0.7.0
• Assignee: Jeff Zhang
• Reporter: Jeff Zhang
• Created: 01/Apr/15 02:11
• Updated: 30/Jun/15 04:53
• Resolved: 23/Apr/15 01:47

Description

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.

        if (this.historyEventHandler.hasRecoveryFailed()) {
          LOG.warn("Recovery had a fatal error, shutting down session after" +
              " DAG completion");
          sessionStopped.set(true);
        }