[GitHub] incubator-griffin pull request #448: [GRIFFIN-208] log exception details whe...

[chemikadze]

Github user chemikadze commented on a diff in the pull request:

    https://github.com/apache/incubator-griffin/pull/448#discussion_r228391307
  
    --- Diff: 
measure/src/main/scala/org/apache/griffin/measure/Application.scala ---
    @@ -104,12 +106,18 @@ object Application extends Loggable {
           case Success(_) =>
             info("process end success")
           case Failure(ex) =>
    -        error(s"process end error: ${ex.getMessage}")
    +        error(s"process end error: ${ex.getMessage}", ex)
             shutdown
             sys.exit(-5)
         }
     
         shutdown
    +
    +    if (success) {
    --- End diff --
    
    removed exit(0), looks like org.apache.spark.deploy.yarn.ApplicationMaster 
is not liking direct exit(0)


---