search

apache / datafusion-ballista

Apache DataFusion Ballista Distributed Query Engine
https://datafusion.apache.org/ballista
Apache License 2.0
1.55k stars 197 forks source link

Update DataFusion to 43 #1125

Open Dandandan opened 2 days ago

Dandandan commented 2 days ago

Which issue does this PR close?

Closes #1124

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Dandandan commented 2 days ago

I can succesfully run the tpc-h queries, let's wait until build is green.

milenkovicm commented 2 days ago

I have mixed feelings about keeping that test which involves ballista python, should we remove it and risk breaking python from time to time? wdyt @Dandandan @andygrove

Epicism commented 2 days ago

PoV: Shouldn't the Python code be in its own repo? Ballista is being redesigned as a library, and Python is an application of the library...

On Wed, Nov 20, 2024 at 4:05 PM Marko Milenković @.***> wrote:

I have mixed feelings about keeping that test which involves ballista python, should we remove it and risk breaking python from time to time? wdyt @Dandandan https://github.com/Dandandan @andygrove https://github.com/andygrove

— Reply to this email directly, view it on GitHub https://github.com/apache/datafusion-ballista/pull/1125#issuecomment-2489539232, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACNEIPJOQLOCMX5DNZEFQGL2BT2P7AVCNFSM6AAAAABSFBQZ3GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOBZGUZTSMRTGI . You are receiving this because you are subscribed to this thread.Message ID: @.***>