Github user jkbradley commented on the issue:

    https://github.com/apache/spark/pull/20633
  
    Sorry for the pause in review.
    LGTM
    Merging with master
    
    @dbtsai  I'm going to merge this since I'm worried it will collect more 
conflicts, but let's discuss more if needed.
    
    @viirya We'll need to update Python's DefaultParamsReader as well for Spark 
2.4 in order to keep it in sync with Scala/Java.  R thankfully should not 
require anything since it only has wrappers.  I'll make & link a JIRA.  Will 
you have time to work on that?
    
    Thanks @viirya !


---

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

Reply via email to