[GitHub] spark pull request: Fix race condition at SchedulerBackend.isReady...

2014-07-22 Thread li-zhihui
GitHub user li-zhihui opened a pull request: https://github.com/apache/spark/pull/1525 Fix race condition at SchedulerBackend.isReady in standalone mode In SPARK-1946(PR #900), configuration codespark.scheduler.minRegisteredExecutorsRatio/code was introduced. However, in

[GitHub] spark pull request: Fix race condition at SchedulerBackend.isReady...

2014-07-22 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/1525#issuecomment-49714817 Can one of the admins verify this patch? --- 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

[GitHub] spark pull request: Fix race condition at SchedulerBackend.isReady...

2014-07-22 Thread li-zhihui
Github user li-zhihui commented on the pull request: https://github.com/apache/spark/pull/1525#issuecomment-49714878 @kayousterhout @tgravescs --- 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: Fix race condition at SchedulerBackend.isReady...

2014-07-22 Thread markhamstra
Github user markhamstra commented on a diff in the pull request: https://github.com/apache/spark/pull/1525#discussion_r15240513 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala --- @@ -47,19 +47,19 @@ class

[GitHub] spark pull request: Fix race condition at SchedulerBackend.isReady...

2014-07-22 Thread markhamstra
Github user markhamstra commented on a diff in the pull request: https://github.com/apache/spark/pull/1525#discussion_r15242266 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/SparkDeploySchedulerBackend.scala --- @@ -108,4 +108,8 @@ private[spark] class

[GitHub] spark pull request: Fix race condition at SchedulerBackend.isReady...

2014-07-22 Thread markhamstra
Github user markhamstra commented on a diff in the pull request: https://github.com/apache/spark/pull/1525#discussion_r15242315 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/SparkDeploySchedulerBackend.scala --- @@ -108,4 +108,8 @@ private[spark] class

[GitHub] spark pull request: Fix race condition at SchedulerBackend.isReady...

2014-07-22 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/1525#issuecomment-49825479 can you please also file a jira for this --- 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

[GitHub] spark pull request: Fix race condition at SchedulerBackend.isReady...

2014-07-22 Thread tgravescs
Github user tgravescs commented on a diff in the pull request: https://github.com/apache/spark/pull/1525#discussion_r15268342 --- Diff: docs/configuration.md --- @@ -707,21 +707,22 @@ Apart from these, the following properties are also available, and may be useful /td

[GitHub] spark pull request: Fix race condition at SchedulerBackend.isReady...

2014-07-22 Thread tgravescs
Github user tgravescs commented on the pull request: https://github.com/apache/spark/pull/1525#issuecomment-49825514 Jenkins, 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

[GitHub] spark pull request: Fix race condition at SchedulerBackend.isReady...

2014-07-22 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/1525#issuecomment-49825654 QA tests have started for PR 1525. This patch merges cleanly. brView progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17006/consoleFull ---

[GitHub] spark pull request: Fix race condition at SchedulerBackend.isReady...

2014-07-22 Thread li-zhihui
Github user li-zhihui commented on a diff in the pull request: https://github.com/apache/spark/pull/1525#discussion_r15268735 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala --- @@ -47,19 +47,19 @@ class

[GitHub] spark pull request: Fix race condition at SchedulerBackend.isReady...

2014-07-22 Thread li-zhihui
Github user li-zhihui commented on a diff in the pull request: https://github.com/apache/spark/pull/1525#discussion_r15268755 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/SparkDeploySchedulerBackend.scala --- @@ -108,4 +108,8 @@ private[spark] class

[GitHub] spark pull request: Fix race condition at SchedulerBackend.isReady...

2014-07-22 Thread li-zhihui
Github user li-zhihui commented on a diff in the pull request: https://github.com/apache/spark/pull/1525#discussion_r15268932 --- Diff: docs/configuration.md --- @@ -707,21 +707,22 @@ Apart from these, the following properties are also available, and may be useful /td

[GitHub] spark pull request: Fix race condition at SchedulerBackend.isReady...

2014-07-22 Thread markhamstra
Github user markhamstra commented on a diff in the pull request: https://github.com/apache/spark/pull/1525#discussion_r15268935 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala --- @@ -47,19 +47,19 @@ class