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

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

beliefer 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_r261137099
 
 

 ##########
 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:
   I think 'the initial set of executors will be at least this large' explains all.

----------------------------------------------------------------
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