Uploaded image for project: 'Apache Tez'
  1. Apache Tez
  2. TEZ-1844

Shouldn't invoke system.exit in local mode when AM is failed to start

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.7.0
    • Component/s: None
    • Labels:
      None
    • Target Version/s:

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

        Attachments

        1. TEZ-1844.patch
          13 kB
          Jeff Zhang
        2. TEZ-1844-2.patch
          11 kB
          Jeff Zhang
        3. TEZ-1844-3.patch
          12 kB
          Jeff Zhang

          Issue Links

            Activity

              People

              • Assignee:
                zjffdu Jeff Zhang
                Reporter:
                zjffdu Jeff Zhang
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: