[GitHub] HyukjinKwon commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events

[GitBox <gi...@apache.org>]

HyukjinKwon commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events
URL: https://github.com/apache/spark/pull/23263#issuecomment-446446483
 
 
   I wanted to avoid a big change (also considering ML side is rather less active) .. but this looked to me necessarily required changes to address this issue fully .. Still there's no breaking change at all but only one addition to the developer API.
   
   I can maybe try to do this for few specific implementations to make the change/affected scope small experimentally. However, my impression is that it's a required change to fully address this issue. I'm still willing to follow any guidance. I would appreciate it.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org