mridulm commented on code in PR #47815: URL: https://github.com/apache/spark/pull/47815#discussion_r1747751422
########## core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala: ########## @@ -167,7 +168,7 @@ private[spark] class DAGScheduler( // Stages that must be resubmitted due to fetch failures private[scheduler] val failedStages = new HashSet[Stage] - private[scheduler] val activeJobs = new HashSet[ActiveJob] + private[spark] val activeJobs = new HashSet[ActiveJob] Review Comment: I am not in favor of exposing this state - even if only for tests currently. Let us keep PR consistent, the change is small enough that we dont need to split it. -- 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]
