[GitHub] spark pull request: [SPARK-13566] Avoid deadlock between BlockMana...

2016-03-10 Thread cenyuhai
Github user cenyuhai commented on a diff in the pull request: https://github.com/apache/spark/pull/11546#discussion_r55668498 --- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala --- @@ -227,6 +228,17 @@ private[spark] class Executor(

[GitHub] spark pull request: [SPARK-13566] Avoid deadlock between BlockMana...

2016-03-10 Thread cenyuhai
Github user cenyuhai commented on a diff in the pull request: https://github.com/apache/spark/pull/11546#discussion_r55664945 --- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala --- @@ -227,6 +228,17 @@ private[spark] class Executor(

[GitHub] spark pull request: [SPARK-13566] Avoid deadlock between BlockMana...

2016-03-07 Thread markhamstra
Github user markhamstra commented on a diff in the pull request: https://github.com/apache/spark/pull/11546#discussion_r55273032 --- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala --- @@ -227,6 +228,17 @@ private[spark] class Executor(

[GitHub] spark pull request: [SPARK-13566] Avoid deadlock between BlockMana...

2016-03-07 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/11546#issuecomment-193433237 Whoops, my bad: I misunderstood this patch because it's missing a description. In the future, please submit a more complete description with your PR., --- If your

[GitHub] spark pull request: [SPARK-13566] Avoid deadlock between BlockMana...

2016-03-07 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/11546#issuecomment-193432765 This isn't opened against `branch-1.6` and isn't cleanly backportable to that branch in its current form. Could you please write up a concise description of the race

[GitHub] spark pull request: [SPARK-13566] Avoid deadlock between BlockMana...

2016-03-06 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11546#issuecomment-192881884 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: [SPARK-13566] Avoid deadlock between BlockMana...

2016-03-06 Thread cenyuhai
GitHub user cenyuhai opened a pull request: https://github.com/apache/spark/pull/11546 [SPARK-13566] Avoid deadlock between BlockManager and Executor Thread Temp patch for branch 1.6, avoid deadlock between BlockManager and Executor Thread. You can merge this pull request into a