Github user rdblue commented on the pull request:
https://github.com/apache/spark/pull/11612#issuecomment-196463934
Thanks @vanzin!
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11612
---
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 vanzin commented on the pull request:
https://github.com/apache/spark/pull/11612#issuecomment-196452256
Merging to master.
---
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
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/11612#issuecomment-195612369
LGTM.
---
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 vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11612#discussion_r55907969
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -265,25 +265,52 @@ private[yarn] class YarnAllocator(
// For
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11612#discussion_r55891228
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -265,25 +265,52 @@ private[yarn] class YarnAllocator(
// For
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11612#issuecomment-194558928
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11612#issuecomment-194558783
**[Test build #52771 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52771/consoleFull)**
for PR 11612 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11612#issuecomment-194558923
Merged build finished. Test PASSed.
---
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11612#issuecomment-194552968
**[Test build #52771 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52771/consoleFull)**
for PR 11612 at commit
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/11612#issuecomment-194551028
ok to test
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11612#issuecomment-194506965
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 user rdblue opened a pull request:
https://github.com/apache/spark/pull/11612
SPARK-13779: Avoid cancelling non-local container requests.
## What changes were proposed in this pull request?
To maximize locality, the YarnAllocator would cancel any requests with a
13 matches
Mail list logo
