dongjoon-hyun commented on a change in pull request #23386: [SPARK-26446][CORE]
Add cachedExecutorIdleTimeout docs at ExecutorAllocationManager
URL: https://github.com/apache/spark/pull/23386#discussion_r244264949
##########
File path: core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala
##########
@@ -57,7 +57,8 @@ import org.apache.spark.util.{Clock, SystemClock,
ThreadUtils, Utils}
* a long time to ramp up under heavy workloads.
*
* The remove policy is simpler: If an executor has been idle for K seconds,
meaning it has not
- * been scheduled to run any tasks, then it is removed.
+ * been scheduled to run any tasks, then it is removed. Note that if an
executor caching any data
+ * blocks will be removed until it has been idle for more then L seconds.
Review comment:
Ur, I know the meaning, but this sentence itself looks unclear to me; `if`,
`until`? Could you make this a simpler?
```
Note that if an executor caching any data blocks will be removed until it
has been idle for more then L seconds.
```
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
