-
Notifications
You must be signed in to change notification settings - Fork 249
chore: support DataFusion 50.3.0 #2605
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.
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2605 +/- ##
============================================
+ Coverage 56.12% 59.16% +3.03%
- Complexity 976 1447 +471
============================================
Files 119 147 +28
Lines 11743 13743 +2000
Branches 2251 2360 +109
============================================
+ Hits 6591 8131 +1540
- Misses 4012 4388 +376
- Partials 1140 1224 +84 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
native/core/Cargo.toml
Outdated
| assertables = "9" | ||
| hex = "0.4.3" | ||
| datafusion-functions-nested = { version = "50.2.0" } | ||
| datafusion-functions-nested = { version="50.3.0" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unintended loss of spaces?
| arrow = { version = "56.0.0", features = ["prettyprint", "ffi", "chrono-tz"] } | ||
| async-trait = { version = "0.1" } | ||
| bytes = { version = "1.10.0" } | ||
| parquet = { version = "=56.2.0", default-features = false, features = ["experimental"] } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But you fixed the missing spaces here!
mbutrovich
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved pending CI, no sense in running CI again for a pair of missing spaces.
Which issue does this PR close?
Closes #2586.
Rationale for this change
What changes are included in this PR?
How are these changes tested?