beliefer commented on code in PR #50020:
URL: https://github.com/apache/spark/pull/50020#discussion_r1981414143
##########
core/src/main/scala/org/apache/spark/BarrierCoordinator.scala:
##########
@@ -55,6 +55,7 @@ private[spark] class BarrierCoordinator(
// fetch result, we shall fix the issue.
private lazy val timer = ThreadUtils.newSingleThreadScheduledExecutor(
"BarrierCoordinator barrier epoch increment timer")
+ private var timerFuture: Option[ScheduledFuture[_]] = None
Review Comment:
It seems the `handleRequest` called many times. The timerFuture an option
cannot hold all the `ScheduledFuture`s
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
