Skip to content

Conversation

@mbutrovich
Copy link
Contributor

Which issue does this PR close?

Closes #2719.

Rationale for this change

What changes are included in this PR?

How are these changes tested?

…date. Remove datafusion-sql dependency to improve build times.
@mbutrovich mbutrovich marked this pull request as draft November 7, 2025 16:45
@mbutrovich
Copy link
Contributor Author

Marking as draft since this is just for testing until DataFusion 51.0.0 crates are available.

@codecov-commenter
Copy link

codecov-commenter commented Nov 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.30%. Comparing base (f09f8af) to head (69bd15d).
⚠️ Report is 714 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2729      +/-   ##
============================================
+ Coverage     56.12%   58.30%   +2.17%     
- Complexity      976     1423     +447     
============================================
  Files           119      162      +43     
  Lines         11743    14143    +2400     
  Branches       2251     2362     +111     
============================================
+ Hits           6591     8246    +1655     
- Misses         4012     4704     +692     
- Partials       1140     1193      +53     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mbutrovich mbutrovich changed the title chore: upgrade to DataFusion 51.0.0 and Arrow-rs 57.0.0 deps: upgrade to DataFusion 51.0.0 and Arrow-rs 57.0.0 Nov 7, 2025
@mbutrovich
Copy link
Contributor Author

mbutrovich commented Nov 19, 2025

DF 51.0.0 crates are available, but we're currently planning to review and merge this after the Comet 0.12.0 release. Keeping this as a draft for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Upgrade to DataFusion 51.0.0

2 participants