Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/8610#issuecomment-178266813
@CodingCat thanks for working on this. Sorry that we haven't had the
bandwidth to review this until now.
For this patch, I think it's better if we just throw an exception if
`spark.task.cpus > spark.executor.cores`. We shouldn't start launching
executors with more cores silently just because `spark.task.cpus` is big. That
is not what I would expect as a user.
---
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]
