Re: [I] Expose global context [datafusion-python]

2025-03-15 Thread via GitHub
timsaucer commented on issue #1045: URL: https://github.com/apache/datafusion-python/issues/1045#issuecomment-2710856896 I understand what you mean by "global context is already exposed" but I mean that it should be treated in the wrapper functions as well so that users get a nice python i

Re: [I] Expose global context [datafusion-python]

2025-03-09 Thread via GitHub
timsaucer commented on issue #1045: URL: https://github.com/apache/datafusion-python/issues/1045#issuecomment-2708991997 No, we have plenty of other points in the code where we us the | operator. Also, that issue is related, but not the topic of this issue. We also have a PR up that resolv

Re: [I] Expose global context [datafusion-python]

2025-03-09 Thread via GitHub
Spaarsh commented on issue #1045: URL: https://github.com/apache/datafusion-python/issues/1045#issuecomment-2708982762 Should make a PR that doesn't use the ```|``` operator? We simply need to use the ```Union``` function instead. I already have that code running locally. -- This is an a

Re: [I] Expose global context [datafusion-python]

2025-03-08 Thread via GitHub
Spaarsh commented on issue #1045: URL: https://github.com/apache/datafusion-python/issues/1045#issuecomment-2708491652 Key Points: 1. ```|``` operator not supported for python < 3.10, anyone pulling the main will not be able to use ```SessionContext``` at all 2. ```global_ctx``` alrea

[I] Expose global context [datafusion-python]

2025-03-08 Thread via GitHub
timsaucer opened a new issue, #1045: URL: https://github.com/apache/datafusion-python/issues/1045 **Is your feature request related to a problem or challenge? Please describe what you are trying to do.** #982 added convenience functions to use a global context for operations like `re