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-4370] [Core] Limit number of Netty cores based on executor size #3155
Conversation
Test build #23050 has started for PR 3155 at commit
|
Test build #23050 has finished for PR 3155 at commit
|
Test PASSed. |
Right now, the NettyBlockTransferService uses the total number of cores on the system as the number of threads and buffer arenas to create. The latter is more troubling – this can lead to significant allocation of extra heap and direct memory in situations where executors are relatively small compared to the whole machine. For instance, on a machine with 32 cores, we will allocate (32 cores * 16MB per arena = 512MB) * 2 for client and server = 1GB direct and heap memory. This can be a huge overhead if you're only using, say, 8 of those cores.
Test build #23271 has started for PR 3155 at commit
|
Test build #23272 has started for PR 3155 at commit
|
Test FAILed. |
Jenkins, retest this please. |
Test build #23274 has started for PR 3155 at commit
|
Test build #23271 has finished for PR 3155 at commit
|
Test PASSed. |
Test build #23272 has finished for PR 3155 at commit
|
Test PASSed. |
Test build #23274 has finished for PR 3155 at commit
|
Test PASSed. |
The changes LGTM. @tnachen can you look at the Mesos related change? |
Merging in master & branch-1.2. Thanks! |
Author: Aaron Davidson <aaron@databricks.com> Closes #3155 from aarondav/conf and squashes the following commits: 7045e77 [Aaron Davidson] Add mesos comment 4770f6e [Aaron Davidson] [SPARK-4370] [Core] Limit number of Netty cores based on executor size (cherry picked from commit b9e1c2e) Signed-off-by: Reynold Xin <rxin@databricks.com>
No description provided.