Github user angolon closed the pull request at:
https://github.com/apache/spark/pull/14933
---
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 user angolon commented on the issue:
https://github.com/apache/spark/pull/14933
Done. Thanks again @vanzin :smile:
---
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
GitHub user angolon opened a pull request:
https://github.com/apache/spark/pull/14933
[SPARK-16533][CORE] - backport driver deadlock fix to 2.0
## What changes were proposed in this pull request?
Backport changes from #14710 and #14925 to 2.0
You can merge this pull request
Github user angolon commented on the issue:
https://github.com/apache/spark/pull/14710
retest 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 this feature
enabled and wishes so
Github user angolon commented on the issue:
https://github.com/apache/spark/pull/14710
...*sigh*
---
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
Github user angolon commented on the issue:
https://github.com/apache/spark/pull/14710
Thanks @vanzin. I'm on mobile at the moment - I'll take care of your nit
when I get back to my desk in a couple of hours.
---
If your project is set up for it, you can reply to this emai
Github user angolon commented on a diff in the pull request:
https://github.com/apache/spark/pull/14710#discussion_r76899230
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala
---
@@ -532,39 +547,53 @@ class
Github user angolon commented on the issue:
https://github.com/apache/spark/pull/14710
Thanks for the feedback, @vanzin - all good points. I'll fix them up.
---
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 pr
Github user angolon commented on the issue:
https://github.com/apache/spark/pull/14710
Hrmm... SparkContextSuite passes all tests for me locally. Any idea what
might be happening here?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user angolon commented on the issue:
https://github.com/apache/spark/pull/14710
Done, sorry!
---
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
GitHub user angolon opened a pull request:
https://github.com/apache/spark/pull/14710
[SPARK-16533][CORE]
## What changes were proposed in this pull request?
This pull request reverts the changes made as a part of #14605, which
simply side-steps the deadlock issue. Instead, I
11 matches
Mail list logo