[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2016-01-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10438 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is ena

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2016-01-04 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-168762487 Merging into master 1.6 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2016-01-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-168570221 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2016-01-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-168570140 **[Test build #48624 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48624/consoleFull)** for PR 10438 at commit [`9ed4377`](https://g

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2016-01-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-168570220 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your projec

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2016-01-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-168562638 **[Test build #48624 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48624/consoleFull)** for PR 10438 at commit [`9ed4377`](https://gi

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2016-01-03 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-168561998 test this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this featur

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-168109729 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your projec

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-168109730 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-168109715 **[Test build #48522 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48522/consoleFull)** for PR 10438 at commit [`9ed4377`](https://g

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-168099414 **[Test build #48522 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48522/consoleFull)** for PR 10438 at commit [`9ed4377`](https://gi

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-167739453 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-167739452 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your projec

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-167739406 **[Test build #48398 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48398/consoleFull)** for PR 10438 at commit [`67611ac`](https://g

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-167723226 **[Test build #48398 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48398/consoleFull)** for PR 10438 at commit [`67611ac`](https://gi

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-28 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-167722251 test this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this featur

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-28 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/10438#discussion_r48515392 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -1709,6 +1709,31 @@ private[spark] object Utils extends Logging { }

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-167676895 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your projec

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-167676897 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-167676871 **[Test build #48374 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48374/consoleFull)** for PR 10438 at commit [`67611ac`](https://g

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-28 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-167664384 **[Test build #48374 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48374/consoleFull)** for PR 10438 at commit [`67611ac`](https://gi

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-24 Thread naveenminchu
Github user naveenminchu commented on a diff in the pull request: https://github.com/apache/spark/pull/10438#discussion_r48418553 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -1709,6 +1709,28 @@ private[spark] object Utils extends Logging { }

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-23 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/10438#discussion_r48389634 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -1709,6 +1709,28 @@ private[spark] object Utils extends Logging { }

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-23 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-166969653 Looks good. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this featur

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-22 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10438#discussion_r48324063 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -1709,6 +1709,28 @@ private[spark] object Utils extends Logging { } /*

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-22 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/10438#discussion_r48324053 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -1709,6 +1709,28 @@ private[spark] object Utils extends Logging { } /*

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-166756420 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-166756419 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your projec

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-166756314 **[Test build #48212 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48212/consoleFull)** for PR 10438 at commit [`55032ea`](https://g

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10438#issuecomment-166731620 **[Test build #48212 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48212/consoleFull)** for PR 10438 at commit [`55032ea`](https://gi

[GitHub] spark pull request: [SPARK-12486] Worker should kill the executors...

2015-12-22 Thread nongli
GitHub user nongli opened a pull request: https://github.com/apache/spark/pull/10438 [SPARK-12486] Worker should kill the executors more forcefully if possible. This patch updates the ExecutorRunner's terminate path to use the new java 8 API to terminate processes more forcefull