Re: [PR] Move the udf module to user_defined [datafusion-python]

2025-05-17 Thread via GitHub
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...

Re: [PR] Move the udf module to user_defined [datafusion-python]

2025-04-27 Thread via GitHub
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

Re: [PR] Move the udf module to user_defined [datafusion-python]

2025-04-25 Thread via GitHub
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

[PR] Move the udf module to user_defined [datafusion-python]

2025-04-25 Thread via GitHub
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 `