[GitHub] spark pull request #20998: [SPARK-23888][CORE] speculative task should not r...

2018-04-11 Thread Ngone51
Github user Ngone51 commented on a diff in the pull request: https://github.com/apache/spark/pull/20998#discussion_r180937626 --- Diff: core/src/test/scala/org/apache/spark/scheduler/TaskSetManagerSuite.scala --- @@ -880,8 +880,8 @@ class TaskSetManagerSuite extends SparkFunSuite

[GitHub] spark pull request #20998: [SPARK-23888][CORE] speculative task should not r...

2018-04-11 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/20998#discussion_r180784912 --- Diff: core/src/test/scala/org/apache/spark/scheduler/TaskSetManagerSuite.scala --- @@ -880,8 +880,8 @@ class TaskSetManagerSuite extends SparkFunSuite

[GitHub] spark pull request #20998: [SPARK-23888][CORE] speculative task should not r...

2018-04-11 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/20998#discussion_r180784374 --- Diff: core/src/test/scala/org/apache/spark/scheduler/TaskSetManagerSuite.scala --- @@ -880,6 +880,59 @@ class TaskSetManagerSuite extends SparkFunSuite

[GitHub] spark pull request #20998: [SPARK-23888][CORE] speculative task should not r...

2018-04-10 Thread Ngone51
Github user Ngone51 commented on a diff in the pull request: https://github.com/apache/spark/pull/20998#discussion_r180614873 --- Diff: core/src/test/scala/org/apache/spark/scheduler/TaskSetManagerSuite.scala --- @@ -880,6 +880,59 @@ class TaskSetManagerSuite extends SparkFunSuite

[GitHub] spark pull request #20998: [SPARK-23888][CORE] speculative task should not r...

2018-04-10 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/20998#discussion_r180443917 --- Diff: core/src/test/scala/org/apache/spark/scheduler/TaskSetManagerSuite.scala --- @@ -880,6 +880,59 @@ class TaskSetManagerSuite extends SparkFunSuite

[GitHub] spark pull request #20998: [SPARK-23888][CORE] speculative task should not r...

2018-04-10 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/20998#discussion_r180439612 --- Diff: core/src/test/scala/org/apache/spark/scheduler/TaskSetManagerSuite.scala --- @@ -880,6 +880,59 @@ class TaskSetManagerSuite extends SparkFunSuite

[GitHub] spark pull request #20998: [SPARK-23888][CORE] speculative task should not r...

2018-04-10 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/20998#discussion_r180439559 --- Diff: core/src/test/scala/org/apache/spark/scheduler/TaskSetManagerSuite.scala --- @@ -880,6 +880,59 @@ class TaskSetManagerSuite extends SparkFunSuite

[GitHub] spark pull request #20998: [SPARK-23888][CORE] speculative task should not r...

2018-04-06 Thread Ngone51
GitHub user Ngone51 opened a pull request: https://github.com/apache/spark/pull/20998 [SPARK-23888][CORE] speculative task should not run on a given host where another attempt is already running on ## What changes were proposed in this pull request? There's a bug in: