Skip to content

Conversation

@timsaucer
Copy link
Contributor

Leaving as draft until datafusion-python 50.0.0 releases.

This is a dependency update for DataFusion and Arrow.

The only change that seems like it is impactful is that the arrow buffers no longer use a minimum capacity of 64 bytes. apache/arrow-rs#7984 appears to be the originating source. This updates the statistics that are generated in the unit test.

@timsaucer
Copy link
Contributor Author

I'll rebase it before moving to ready. The first two commits are already in main.

@codecov-commenter
Copy link

codecov-commenter commented Sep 19, 2025

Codecov Report

❌ Patch coverage is 59.09091% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.44%. Comparing base (c6eb6c0) to head (2171dea).

Files with missing lines Patch % Lines
rust/lance-datafusion/src/planner.rs 11.11% 7 Missing and 1 partial ⚠️
rust/lance/src/io/exec/optimizer.rs 83.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4778      +/-   ##
==========================================
+ Coverage   80.88%   81.44%   +0.55%     
==========================================
  Files         323      323              
  Lines      127646   125636    -2010     
  Branches   127646   125636    -2010     
==========================================
- Hits       103251   102319     -932     
+ Misses      20758    19888     -870     
+ Partials     3637     3429     -208     
Flag Coverage Δ
unittests 81.44% <59.09%> (+0.55%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@timsaucer
Copy link
Contributor Author

Closing in favor of #4751

@timsaucer timsaucer closed this Sep 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants