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

• 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: 12/Dec/14 04:20
• Updated: 30/Jun/15 04:52
• Resolved: 24/Dec/14 02:59

Description

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.

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

Issue Links

• relates to: TEZ-684 Uber/Local modes for Tez