[GitHub] spark pull request #15009: [SPARK-17443] Stop Spark Application if launcher ...

2016-09-08 Thread kishorvpatil
Github user kishorvpatil commented on a diff in the pull request: https://github.com/apache/spark/pull/15009#discussion_r78066827 --- Diff: core/src/main/scala/org/apache/spark/launcher/LauncherBackend.scala --- @@ -29,26 +31,63 @@ import org.apache.spark.util.{ThreadUtils, Utils}

[GitHub] spark pull request #15009: [SPARK-17443] Stop Spark Application if launcher ...

2016-09-08 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/15009#discussion_r78060532 --- Diff: core/src/main/scala/org/apache/spark/launcher/LauncherBackend.scala --- @@ -71,6 +110,9 @@ private[spark] abstract class LauncherBackend {

[GitHub] spark pull request #15009: [SPARK-17443] Stop Spark Application if launcher ...

2016-09-08 Thread kishorvpatil
Github user kishorvpatil commented on a diff in the pull request: https://github.com/apache/spark/pull/15009#discussion_r78050727 --- Diff: core/src/main/scala/org/apache/spark/launcher/LauncherBackend.scala --- @@ -71,6 +110,9 @@ private[spark] abstract class LauncherBackend {

[GitHub] spark pull request #15009: [SPARK-17443] Stop Spark Application if launcher ...

2016-09-08 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/15009#discussion_r78018595 --- Diff: core/src/main/scala/org/apache/spark/launcher/LauncherBackend.scala --- @@ -29,26 +31,63 @@ import org.apache.spark.util.{ThreadUtils, Utils}

[GitHub] spark pull request #15009: [SPARK-17443] Stop Spark Application if launcher ...

2016-09-08 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/15009#discussion_r78009842 --- Diff: launcher/src/main/java/org/apache/spark/launcher/AbstractSparkAppHandle.java --- @@ -0,0 +1,138 @@ +/* + * Licensed to the Apache Softwar

[GitHub] spark pull request #15009: [SPARK-17443] Stop Spark Application if launcher ...

2016-09-08 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/15009#discussion_r78010270 --- Diff: launcher/src/main/java/org/apache/spark/launcher/AbstractSparkAppHandle.java --- @@ -0,0 +1,138 @@ +/* + * Licensed to the Apache Softwar

[GitHub] spark pull request #15009: [SPARK-17443] Stop Spark Application if launcher ...

2016-09-08 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/15009#discussion_r78009075 --- Diff: core/src/main/scala/org/apache/spark/launcher/LauncherBackend.scala --- @@ -71,6 +110,9 @@ private[spark] abstract class LauncherBackend {

[GitHub] spark pull request #15009: [SPARK-17443] Stop Spark Application if launcher ...

2016-09-08 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/15009#discussion_r78006275 --- Diff: core/src/main/scala/org/apache/spark/launcher/LauncherBackend.scala --- @@ -29,26 +31,63 @@ import org.apache.spark.util.{ThreadUtils, Utils}

[GitHub] spark pull request #15009: [SPARK-17443] Stop Spark Application if launcher ...

2016-09-08 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/15009#discussion_r78005678 --- Diff: core/src/main/scala/org/apache/spark/launcher/LauncherBackend.scala --- @@ -29,26 +31,63 @@ import org.apache.spark.util.{ThreadUtils, Utils}

[GitHub] spark pull request #15009: [SPARK-17443] Stop Spark Application if launcher ...

2016-09-08 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/15009#discussion_r78005596 --- Diff: core/src/main/scala/org/apache/spark/launcher/LauncherBackend.scala --- @@ -29,26 +31,63 @@ import org.apache.spark.util.{ThreadUtils, Utils}

[GitHub] spark pull request #15009: [SPARK-17443] Stop Spark Application if launcher ...

2016-09-08 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/15009#discussion_r78005411 --- Diff: core/src/main/scala/org/apache/spark/launcher/LauncherBackend.scala --- @@ -29,26 +31,63 @@ import org.apache.spark.util.{ThreadUtils, Utils}

[GitHub] spark pull request #15009: [SPARK-17443] Stop Spark Application if launcher ...

2016-09-08 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/15009#discussion_r78005004 --- Diff: core/src/main/scala/org/apache/spark/launcher/LauncherBackend.scala --- @@ -29,26 +31,63 @@ import org.apache.spark.util.{ThreadUtils, Utils}

[GitHub] spark pull request #15009: [SPARK-17443] Stop Spark Application if launcher ...

2016-09-07 Thread kishorvpatil
GitHub user kishorvpatil opened a pull request: https://github.com/apache/spark/pull/15009 [SPARK-17443] Stop Spark Application if launcher goes down and use reflection ## What changes were proposed in this pull request? 1. `SparkLauncher` provides `stopIfInterrupted` metho