Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19741#discussion_r151308593
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/scheduler/cluster/YarnSchedulerBackend.scala
---
@@ -268,8 +268,13 @@ private[spark] abstract class YarnSchedulerBackend(
logWarning(reason.toString)
driverEndpoint.ask[Boolean](r).onFailure {
--- End diff --
IIRC there are two different `RemoveExecutor` messages in the code, as
confusing as that may be.
But if this one is used in multiple places then it's probably not worth
changing right now, unless you're up for verifying the return value is not
needed in the other places.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
