Github user srowen commented on the issue:
https://github.com/apache/spark/pull/19919
No, this change breaks the existing logic. These instances do not actually
control the default number of instances, but instead are used to detect whether
the value was set at all. With this change, for example, you'd now get an error
if you set min executors to 0 but don't set the number of executor instances
(which you wouldn't normally).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
