Github user nickbp commented on a diff in the pull request:
https://github.com/apache/spark/pull/21516#discussion_r200463853
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosClusterScheduler.scala
---
@@ -122,8 +122,9 @@ private[spark] class MesosClusterScheduler(
conf: SparkConf)
extends Scheduler with MesosSchedulerUtils {
var frameworkUrl: String = _
+ private val metricsSource = new MesosClusterSchedulerSource(this)
--- End diff --
This is just moving the prior initialization from L308 slightly earlier.
This is just being done to make the `sourceName` member available below (avoid
duplicate `mesos_cluster` constants across different files).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
