tgravescs commented on a change in pull request #27843:
URL: https://github.com/apache/spark/pull/27843#discussion_r441549045
##########
File path:
resource-managers/yarn/src/main/scala/org/apache/spark/scheduler/cluster/YarnSchedulerBackend.scala
##########
@@ -67,6 +66,13 @@ private[spark] abstract class YarnSchedulerBackend(
private implicit val askTimeout = RpcUtils.askRpcTimeout(sc.conf)
+ /**
+ * Declare implicit single thread execution context for futures
doRequestTotalExecutors and
+ * doKillExecutors below, avoiding using the global execution context that
may cause conflict
+ * with user code's execution of futures */
Review comment:
nit put the last */ on the next line.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
