lance
lance copied to clipboard
refactor: remove custom ProjectionExec
We implemented our own custom projection ExecutionPlan that took a schema to project. With the latest DataFusion functions (particularly named_struct) we can now express this in terms of the built-in ProjectionExec. This reduces the maintenance burden on us and ensures the equivalents properties and optimizations rules from DataFusion work well without much effort on our part.
Codecov Report
Attention: Patch coverage is 90.90909% with 23 lines in your changes missing coverage. Please review.
Project coverage is 79.20%. Comparing base (
501a642) to head (a68499d).
Additional details and impacted files
@@ Coverage Diff @@
## main #2608 +/- ##
==========================================
+ Coverage 79.14% 79.20% +0.06%
==========================================
Files 218 218
Lines 63617 63731 +114
Branches 63617 63731 +114
==========================================
+ Hits 50351 50481 +130
+ Misses 10325 10309 -16
Partials 2941 2941
| Flag | Coverage Δ | |
|---|---|---|
| unittests | 79.20% <90.90%> (+0.06%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.