Conversation
This improves build times for large structs by only using the arrow_* crates that are required.
Please double check the following review of the pull request:
Changes in the diff
Identified IssuesNo critical or medium issues found. The changes mostly involve refactoring imports and updating dependencies, which are consistent and correct. SummaryThe PR restructures the crate to use the new modular Arrow crates instead of the monolithic The only minor note is a readability improvement: the private module Suggested missing testsNo new features or logic changes were introduced, so no new tests are strictly required. Existing tests were updated to use new imports and they cover serialization/deserialization thoroughly. ConclusionThis PR is a clean and well-executed modernization to the new Arrow Rust modular crates. It improves maintainability and aligns with the Arrow Rust ecosystem's direction. I recommend merging after CI passes. Summon me to re-review when updated! Yours, Gooroo.dev |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #40 +/- ##
==========================================
+ Coverage 94.03% 94.04% +0.01%
==========================================
Files 10 10
Lines 2045 2049 +4
==========================================
+ Hits 1923 1927 +4
Misses 122 122 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This improves build times for large structs by only using the arrow_* crates that are required.