-
Couldn't load subscription status.
- Fork 537
chore: upgrade to delta-kernel-rs 0.16.0 and remove more dependencies #3773
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❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #3773 +/- ##
==========================================
+ Coverage 76.06% 76.09% +0.02%
==========================================
Files 145 145
Lines 45025 45117 +92
Branches 45025 45117 +92
==========================================
+ Hits 34249 34330 +81
- Misses 9094 9098 +4
- Partials 1682 1689 +7 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
8d6ceeb to
c046b72
Compare
2ac42d6 to
b1c7357
Compare
e60e8ae to
827c3de
Compare
There were a number of symbols only used during `datafusion` usage Signed-off-by: R. Tyler Croy <[email protected]>
Signed-off-by: R. Tyler Croy <[email protected]>
This dependency comes from the chrono org, so I kinda trust it, but it's not something we should have in the hierarchy unless it's being used Signed-off-by: R. Tyler Croy <[email protected]>
The StructType public API change was pretty invasive, oof. Fixes delta-io#3774 Signed-off-by: R. Tyler Croy <[email protected]>
The kernel level is doing this validation now Signed-off-by: R. Tyler Croy <[email protected]>
827c3de to
d47e7dc
Compare
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.
👍
Supply chain attacks are on my mind lately for no good reason whatsoever, so
taking some time to purge unnecessary dependencies from our tree where I can
find it.