srowen commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-454845552 It's not a big change, yeah. One concern had been whether this duplicated the existing ML instrumentation. I see that this uses the same instrumentation mechanism now, right? That's good. The current instrumentation generally just logs info to the log. The new MLEvents don't do anything by default, if I read this correctly. Would it make sense to cause them to log messages by default?
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
