Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8007#discussion_r38608209
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -211,8 +218,11 @@ private[yarn] class YarnAllocator(
* Deal with any containers YARN has granted to us by possibly launching
executors in them.
*
* This must be synchronized because variables read in this method are
mutated by other methods.
+ *
+ * Returns a list of executor loss reasons discovered by the allocator,
which can then be
+ * forwarded to the driver by the calling ApplicationMaster.
*/
- def allocateResources(): Unit = synchronized {
+ def allocateResources(): Map[String, ExecutorLossReason] = synchronized {
--- End diff --
this doesn't need to change doe sit? Can you revert it?
---
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]
