Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/21078#discussion_r182834828
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/util/Instrumentation.scala ---
@@ -150,3 +156,35 @@ private[spark] object Instrumentation {
}
}
+
+private[spark] class OptionalInstrument private (
+ val instrument: Option[Instrumentation[_ <: Estimator[_]]],
+ val className: String) extends Logging {
+
+ def this(instr: Instrumentation[_ <: Estimator[_]]) = this(Some(instr),
"")
--- End diff --
Also, let's set logName in the constructor so that, when we create() this
with an Instrumentation instance, we can match the Instrumentation instance's
logName: instr.getClass.getName.stripSuffix(...)
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
