Github user HyukjinKwon commented on the issue:

    https://github.com/apache/spark/pull/20137
  
    > BTW, this PR is not just for `asNondeterministic()`. We have the same 
issue for the deterministic UDFs.
    
    Yup, the fix for deterministic UDFs seem fine but the change about 
`asNondeterministic()` bugs me.
    
    If you meant the docstring about `asNondeterministic` itself (not the 
wrapped function instance as above),
    
    I think we can do the things below:
    
    ```python
           wrapper.asNondeterministic = functools.wraps(
               self.asNondeterministic)(lambda: 
self.asNondeterministic()._wrapped())
    ```



---

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

Reply via email to