[GitHub] spark pull request #22202: [SPARK-25211][Core] speculation and fetch failed ...

2018-08-24 Thread liutang123
Github user liutang123 commented on a diff in the pull request: https://github.com/apache/spark/pull/22202#discussion_r212601673 --- Diff: core/src/test/scala/org/apache/spark/scheduler/DAGSchedulerSuite.scala --- @@ -2246,58 +2247,6 @@ class DAGSchedulerSuite extends SparkFunSuite

[GitHub] spark pull request #22202: [SPARK-25211][Core] speculation and fetch failed ...

2018-08-23 Thread xuanyuanking
Github user xuanyuanking commented on a diff in the pull request: https://github.com/apache/spark/pull/22202#discussion_r212365264 --- Diff: core/src/test/scala/org/apache/spark/scheduler/DAGSchedulerSuite.scala --- @@ -2246,58 +2247,6 @@ class DAGSchedulerSuite extends SparkFunSui

[GitHub] spark pull request #22202: [SPARK-25211][Core] speculation and fetch failed ...

2018-08-23 Thread liutang123
GitHub user liutang123 opened a pull request: https://github.com/apache/spark/pull/22202 [SPARK-25211][Core] speculation and fetch failed result in hang of job ## What changes were proposed in this pull request? In current `DAGScheduler.handleTaskCompletion` code, when a shu