Fix app status reporting.

Author: Marcelo Vanzin

yarn/common/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala

@@ -128,6 +128,7 @@ private[spark] class ApplicationMaster(args: ApplicationMasterArguments,
       logInfo(s"Finishing ApplicationMaster with $status"  +
         Option(diagnostics).map(msg => s" (diag message: $msg)").getOrElse(""))
       finished = true
+      finalStatus = status
       reporterThread.interrupt()
       try {
         if (Thread.currentThread() != reporterThread) {
@@ -341,17 +342,19 @@ private[spark] class ApplicationMaster(args: ApplicationMasterArguments,
 
     val t = new Thread {
       override def run() {
+        var status = FinalApplicationStatus.FAILED
         try {
           // Copy
           val mainArgs = new Array[String](args.userArgs.size)
           args.userArgs.copyToArray(mainArgs, 0, args.userArgs.size)
           mainMethod.invoke(null, mainArgs)
           // Some apps have "System.exit(0)" at the end.  The user thread will stop here unless
           // it has an uncaught exception thrown out.  It needs a shutdown hook to set SUCCEEDED.
-          finalStatus = FinalApplicationStatus.SUCCEEDED
+          status = FinalApplicationStatus.SUCCEEDED
         } finally {
           logDebug("Finishing main")
         }
+        finalStatus = status
       }
     }
     t.setName("Driver")