LuciferYang commented on a change in pull request #25309: [SPARK-28577][YARN]Resource capability requested for each executor add offHeapMemorySize URL: https://github.com/apache/spark/pull/25309#discussion_r318525158
########## File path: docs/configuration.md ########## @@ -250,17 +250,17 @@ of the most common options to set are: <td><code>spark.executor.memoryOverhead</code></td> <td>executorMemory * 0.10, with minimum of 384 </td> <td> - Amount of non-heap memory to be allocated per executor process in cluster mode, in MiB unless + Amount of additional memory to be allocated per executor process in cluster mode, in MiB unless Review comment: maybe we should remove `interned strings`, it use heap space after Java8 ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
