[GitHub] spark pull request: SPARK-3612. Executor shouldn't quit if heartbe...

2014-09-23 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/2487#issuecomment-56586630 Thanks - my bad for not catching this in the review. IIRC we directly discussed making sure this didn't happen, but we must have missed this specific case. --- If

[GitHub] spark pull request: SPARK-3612. Executor shouldn't quit if heartbe...

2014-09-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/2487 --- 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

[GitHub] spark pull request: SPARK-3612. Executor shouldn't quit if heartbe...

2014-09-22 Thread sryza
GitHub user sryza opened a pull request: https://github.com/apache/spark/pull/2487 SPARK-3612. Executor shouldn't quit if heartbeat message fails to reach ... ...the driver You can merge this pull request into a Git repository by running: $ git pull

[GitHub] spark pull request: SPARK-3612. Executor shouldn't quit if heartbe...

2014-09-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2487#issuecomment-56341369 [QA tests have started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/20648/consoleFull) for PR 2487 at commit

[GitHub] spark pull request: SPARK-3612. Executor shouldn't quit if heartbe...

2014-09-22 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/2487#issuecomment-56341801 LGTM pending Jenkins. --- 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

[GitHub] spark pull request: SPARK-3612. Executor shouldn't quit if heartbe...

2014-09-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/2487#issuecomment-56347933 [QA tests have finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/20648/consoleFull) for PR 2487 at commit