GitHub user wangyum opened a pull request:
https://github.com/apache/spark/pull/23030
[MINOR][YARN] Make memLimitExceededLogMessage more clean
## What changes were proposed in this pull request?
Current `memLimitExceededLogMessage`:
<img
src="https://user-images.githubusercontent.com/5399861/48467789-ec8e1000-e824-11e8-91fc-280d342e1bf3.png";
width="360">
Itâs not very clear, because physical memory exceeds but suggestion
contains virtual memory config. This pr makes it more clear and replace
deprecated config: ```spark.yarn.executor.memoryOverhead```.
## How was this patch tested?
manual tests
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/wangyum/spark EXECUTOR_MEMORY_OVERHEAD
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/23030.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #23030
----
commit e7ff43d0056e38033ae2a2edf81b296797bb46e0
Author: Yuming Wang <yumwang@...>
Date: 2018-11-14T07:46:58Z
Make memLimitExceededLogMessage more clean
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
![https://user-images.githubusercontent.com/5399861/48467789-ec8e1000-e824-11e8-91fc-280d342e1bf3.png"](https://user-images.githubusercontent.com/5399861/48467789-ec8e1000-e824-11e8-91fc-280d342e1bf3.png"){kind=link}