Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/7532#issuecomment-125303427
@CodingCat `requestTotalExecutor` is never meant to be called by the user.
All of this happens if you enable dynamic allocation in the background. The
only reason we have `requestTotalExecutors` in the first place is for canceling
pending requests, not for killing executors.
Also, as I mentioned, this discussion is completely irrelevant to this
patch since the API was added a long time ago. If you wish to suggest something
different from the existing design, we should do it through a new issue on
JIRA, though I personally don't think it is an important one since it's all
internal.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
