You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2018/12/17 09:09:49 UTC

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

HyukjinKwon commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline
URL: https://github.com/apache/spark/pull/23263#issuecomment-447771837
 
 
   I made the change smaller by only sending events at `Pipeline`s. So, it should really be conservative enough in terms of being intrusive ...
   
   I preserved previous proposal at another branch (https://github.com/apache/spark/compare/master...HyukjinKwon:SPARK-23674-2?expand=1), which adds events support for all implementations within ML. I can propose this in the future when we see some values on 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