srowen commented on a change in pull request #23907: [SPARK-27003] [MINOR][DOC]Default value of spark.executor.instances is applicable… URL: https://github.com/apache/spark/pull/23907#discussion_r261234058
########## File path: docs/running-on-yarn.md ########## @@ -270,7 +270,7 @@ To use a custom metrics.properties for the application master and executors, upd <td><code>spark.executor.instances</code></td> <td><code>2</code></td> <td> - The number of executors for static allocation. With <code>spark.dynamicAllocation.enabled</code>, the initial set of executors will be at least this large. + The number of executors for static allocation. With <code>spark.dynamicAllocation.enabled</code>, the initial set of executors will be at least this large. Default value for static allocation is 2, whereas in dynamic allocation, is 0. Review comment: It actually defaults to the min number of executors. This is already documented on this page, as is the default of 2 here. I wouldn't add this. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
