Github user xuanyuanking commented on the issue:

    https://github.com/apache/spark/pull/21370
  
    ```
    Test coverage is the most critical when we refactor the existing code and 
add new features. Hopefully, when you submit new PRs in the future, could you 
also improve this part?
    ```
    Of cause, I'll do this in a follow up PR and answer all question from Xiao 
this night. Thanks for all your comments.


---

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

Reply via email to