Re: [PR] feat: reads using global ctx [datafusion-python]

2025-03-08 Thread via GitHub
timsaucer commented on PR #982: URL: https://github.com/apache/datafusion-python/pull/982#issuecomment-2708552064 Thanks, Kyle. More generally I’ll see about the impact of turning on all the rules and then removing a few specifically as needed -- This is an automated message from the Apa

Re: [PR] feat: reads using global ctx [datafusion-python]

2025-03-08 Thread via GitHub
kylebarron commented on PR #982: URL: https://github.com/apache/datafusion-python/pull/982#issuecomment-2708499224 There needs to be an initial import `from __future__ import annotations` on the first line to be able to use `|` typing syntax. There are ruff rules to check this and we

Re: [PR] feat: reads using global ctx [datafusion-python]

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

Re: [PR] feat: reads using global ctx [datafusion-python]

2025-03-08 Thread via GitHub
timsaucer merged PR #982: URL: https://github.com/apache/datafusion-python/pull/982 -- 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: reads using global ctx [datafusion-python]

2025-03-08 Thread via GitHub
timsaucer commented on PR #982: URL: https://github.com/apache/datafusion-python/pull/982#issuecomment-2708291557 I added a few lines to the documentation, rebased, and applied updated ruff formatting. -- This is an automated message from the Apache Git Service. To respond to the message

Re: [PR] feat: reads using global ctx [datafusion-python]

2025-01-06 Thread via GitHub
kylebarron commented on code in PR #982: URL: https://github.com/apache/datafusion-python/pull/982#discussion_r1904778623 ## python/datafusion/io.py: ## @@ -0,0 +1,181 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. Se

Re: [PR] feat: reads using global ctx [datafusion-python]

2025-01-06 Thread via GitHub
kevinjqliu commented on code in PR #982: URL: https://github.com/apache/datafusion-python/pull/982#discussion_r1904776793 ## python/datafusion/io.py: ## @@ -0,0 +1,181 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. Se

Re: [PR] feat: reads using global ctx [datafusion-python]

2025-01-06 Thread via GitHub
kylebarron commented on code in PR #982: URL: https://github.com/apache/datafusion-python/pull/982#discussion_r1904717182 ## python/datafusion/io.py: ## @@ -0,0 +1,181 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. Se