Github user BryanCutler commented on the issue:

    https://github.com/apache/spark/pull/20295
  
    > @BryanCutler yeah I think kwargs is another option. But I think the API 
in this PR is more consistent with the exsiting APIs though.
    
    Yeah if it's consistent with other APIs then sounds fine with me.  My 
concern was in giving the user too many options that it starts to get confusing 
to make UDFs. If it's a familiar API then that probably won't be the case.


---

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

Reply via email to