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

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

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

 ##########
 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:
   @beliefer The default value 2 is not applicable for dynamic allocation case.
   
   That is, if user doesn't set spark.executor.instances, then it takes 0 as the value, in case of dynamic allocation. 

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