Re: [PR] feat: support enable_url_table config [datafusion-python]

2025-01-08 Thread via GitHub
timsaucer merged PR #980: URL: https://github.com/apache/datafusion-python/pull/980 -- 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...@d

Re: [PR] feat: support enable_url_table config [datafusion-python]

2025-01-08 Thread via GitHub
timsaucer commented on code in PR #980: URL: https://github.com/apache/datafusion-python/pull/980#discussion_r1907894122 ## python/datafusion/context.py: ## @@ -472,6 +472,18 @@ def __init__( self.ctx = SessionContextInternal(config, runtime) +def enable_url_tab

Re: [PR] feat: support enable_url_table config [datafusion-python]

2025-01-08 Thread via GitHub
timsaucer commented on PR #980: URL: https://github.com/apache/datafusion-python/pull/980#issuecomment-2578313618 Very nice. Thank you. I've kicked off CI and will merge if all goes through. -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] feat: support enable_url_table config [datafusion-python]

2025-01-07 Thread via GitHub
timsaucer commented on PR #980: URL: https://github.com/apache/datafusion-python/pull/980#issuecomment-2575307105 Thank you for the addition! Would it be simpler to just expose the function on the session context instead of making it part of the initialization? -- This is an automated me

Re: [PR] feat: support enable_url_table config [datafusion-python]

2025-01-07 Thread via GitHub
chenkovsky commented on PR #980: URL: https://github.com/apache/datafusion-python/pull/980#issuecomment-2575455417 > Thank you for the addition! Would it be simpler to just expose the function on the session context instead of making it part of the initialization? Even if you do think it's

[PR] feat: support enable_url_table config [datafusion-python]

2024-12-21 Thread via GitHub
chenkovsky opened a new pull request, #980: URL: https://github.com/apache/datafusion-python/pull/980 # Which issue does this PR close? Closes #. # Rationale for this change without url table support, we have to create table, if we want to query a file. # What ch