[GitHub] srowen commented on a change in pull request #23907: [SPARK-27003] [MINOR][DOC]Default value of spark.executor.instances is applicable…

[GitBox <gi...@apache.org>]

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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org