review comment

tejasapatil · tejasapatil · commit 54990718ec5c · 2016-07-15T10:29:36.000-07:00
diff --git a/core/src/main/scala/org/apache/spark/executor/CoarseGrainedExecutorBackend.scala b/core/src/main/scala/org/apache/spark/executor/CoarseGrainedExecutorBackend.scala
@@ -149,7 +149,11 @@ private[spark] class CoarseGrainedExecutorBackend(
    * back-end may not want to take the parent process down.
    */
   protected def exitExecutor(code: Int, reason: String, throwable: Throwable = null) = {
-    logError(reason, throwable)
+    if (throwable == null) {
+      logError(reason)
+    } else {
+      logError(reason, throwable)
+    }
     System.exit(code)
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,11 @@ private[spark] class CoarseGrainedExecutorBackend(`
`149`	`149`	`* back-end may not want to take the parent process down.`
`150`	`150`	`*/`
`151`	`151`	`protected def exitExecutor(code: Int, reason: String, throwable: Throwable = null) = {`
`152`		`- logError(reason, throwable)`
	`152`	`+ if (throwable == null) {`
	`153`	`+ logError(reason)`
	`154`	`+ } else {`
	`155`	`+ logError(reason, throwable)`
	`156`	`+ }`
`153`	`157`	`System.exit(code)`
`154`	`158`	`}`
`155`	`159`	`}`