Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-2774] Set preferred locations for reduce tasks #4576

Closed
wants to merge 3 commits into from

Conversation

shivaram
Copy link
Contributor

This is another attempt at #1697 addressing some of the earlier concerns.
This adds a couple of thresholds based on number map and reduce tasks beyond which we don't use preferred locations for reduce tasks.

This patch also fixes some bugs in DAGSchedulerSuite where the MapStatus objects created didn't have the right number of reducers set.

cc @JoshRosen @rxin @pwendell

This is another attempt at apache#1697 addressing some of the earlier concerns.
This adds a couple of thresholds based on number map and reduce tasks
beyond which we don't use preferred locations for reduce tasks.

This patch also fixes some bugs in DAGSchedulerSuite where the MapStatus
objects created didn't have the right number of reducers set.
@SparkQA
Copy link

SparkQA commented Feb 12, 2015

Test build #27386 has started for PR 4576 at commit 34d0283.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 12, 2015

Test build #27386 has finished for PR 4576 at commit 34d0283.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27386/
Test FAILed.

@SparkQA
Copy link

SparkQA commented Feb 13, 2015

Test build #27398 has started for PR 4576 at commit 774751b.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 13, 2015

Test build #27398 has finished for PR 4576 at commit 774751b.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27398/
Test PASSed.

@shivaram
Copy link
Contributor Author

Ping

@pwendell
Copy link
Contributor

pwendell commented Jun 4, 2015

I am cleaning up old PR's and would propose to close this issue pending further discussion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants