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-9244] Increase some memory defaults #7586

Closed
wants to merge 1 commit into from

Conversation

mateiz
Copy link
Contributor

@mateiz mateiz commented Jul 22, 2015

There are a few memory limits that people hit often and that we could
make higher, especially now that memory sizes have grown.

  • spark.akka.frameSize: This defaults at 10 but is often hit for map
    output statuses in large shuffles. This memory is not fully allocated
    up-front, so we can just make this larger and still not affect jobs
    that never sent a status that large. We increase it to 128.
  • spark.executor.memory: Defaults at 512m, which is really small. We
    increase it to 1g.

There are a few memory limits that people hit often and that we could
make higher, especially now that memory sizes have grown.

- spark.akka.frameSize: This defaults at 10 but is often hit for map
  output statuses in large shuffles. This memory is not fully allocated
  up-front, so we can just make this larger and still not affect jobs
  that never sent a status that large. We increase it to 128.

- spark.executor.memory: Defaults at 512m, which is really small. We
  increase it to 1g.
@SparkQA
Copy link

SparkQA commented Jul 22, 2015

Test build #38039 timed out for PR 7586 at commit ce0038a after a configured wait of 175m.

@mateiz
Copy link
Contributor Author

mateiz commented Jul 22, 2015

retest this please

@SparkQA
Copy link

SparkQA commented Jul 22, 2015

Test build #38086 has finished for PR 7586 at commit ce0038a.

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

@SparkQA
Copy link

SparkQA commented Jul 22, 2015

Test build #1159 has finished for PR 7586 at commit ce0038a.

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

@srowen
Copy link
Member

srowen commented Jul 22, 2015

FWIW LGTM

@mateiz
Copy link
Contributor Author

mateiz commented Jul 22, 2015

Alright, thanks for taking a look.

@asfgit asfgit closed this in fe26584 Jul 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants