Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21347
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/21347#discussion_r189057734
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/util/Instrumentation.scala ---
@@ -125,6 +125,19 @@ private[spark] class Instrumentation[E <:
Estimato
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/21347#discussion_r188820444
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/util/Instrumentation.scala ---
@@ -125,6 +125,19 @@ private[spark] class Instrumentation[E <:
Estimator[_
GitHub user ludatabricks opened a pull request:
https://github.com/apache/spark/pull/21347
[SPARK-24290][ML] add support for Array input for
instrumentation.logNamedValue
## What changes were proposed in this pull request?
Extend instrumentation.logNamedValue to support Arr