Github user HyukjinKwon commented on the issue:

    https://github.com/apache/spark/pull/21320
  
    @mallman, can you close this one and put some changes into @ajacques branch 
then? no point of opening duplicated changes. Since @ajacques is at least 
willing to address other comments including styles in 
https://github.com/apache/spark/pull/21889, we will go with that PR in any 
event. Right?
    
    If you are willing to update this PR, please address style comments here 
too. Otherwise, let me just close this and deduplicate the effort. What's the 
point of leaving this PR open?


---

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

Reply via email to