timsaucer merged PR #1112:
URL: https://github.com/apache/datafusion-python/pull/1112
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...
crystalxyz commented on PR #1112:
URL:
https://github.com/apache/datafusion-python/pull/1112#issuecomment-2833418001
@timsaucer The documentation changes look good to me! Do you think that
adding a comment in `python/datafusion/user_defined` to explain the renaming
would be helpful? People
timsaucer commented on PR #1112:
URL:
https://github.com/apache/datafusion-python/pull/1112#issuecomment-2831187602
@crystalxyz FYI this does touch on the documentation you wrote. Please let
me know if you think I should change back the formatting. I did need to switch
from markdown to rst
timsaucer opened a new pull request, #1112:
URL: https://github.com/apache/datafusion-python/pull/1112
# Which issue does this PR close?
None
# Rationale for this change
Right now we have some minor confusion when importing from the `udf` module
because at the root of `