Github user jkbradley commented on the issue:

    https://github.com/apache/spark/pull/21344
  
    LGTM
    It'll be nice when we move the implementation to spark.ml in the future so 
that we can log more info, but this is good for now.
    Thanks @MrBago and @ludatabricks !
    
    Merging with master


---

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

Reply via email to