Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update dependencies #129

Merged
merged 2 commits into from
Oct 9, 2024
Merged

Update dependencies #129

merged 2 commits into from
Oct 9, 2024
+19 −175

Conversation

ccciudatu
Copy link
Collaborator

arrow 53.1.0
datafusion 42.0.0

@ccciudatu
Update dependencies
2f333df 
arrow 53.1.0
datafusion 42.0.0
@ccciudatu
Copy link
Collaborator Author

@phillipleblanc Hmm, just noticed the upgrade is also part of #112.
Won't close this though, as it also switches to the upstream flight sql client (and removes the copy/pasted code). So I'll rebase once the other PR is merged (unless we find any reason to merge this one first).

@phillipleblanc
Copy link
Collaborator

@ccciudatu we can merge this first. The other PR is waiting on duckdb-rs to release a new version, which may take a while.

@ccciudatu ccciudatu merged commit aacc461 into datafusion-contrib:main Oct 9, 2024
3 checks passed
@ccciudatu ccciudatu deleted the arrow-53 branch October 9, 2024 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@phillipleblanc phillipleblanc phillipleblanc approved these changes

@sgrebnov sgrebnov Awaiting requested review from sgrebnov

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ccciudatu @phillipleblanc