Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20837#discussion_r174995625
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/util/Instrumentation.scala ---
@@ -55,6 +55,11 @@ private[spark] class Instrumentation[E <: Estimator[_]]
private (
s" storageLevel=${dataset.getStorageLevel}")
}
+ override def logWarning(msg: => String): Unit = {
+ logNamedValue(Instrumentation.loggerTags.warning, msg)
+ super.logWarning(msg)
--- End diff --
I doubt it will log repeatedly here.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
