fix: version binary search

[aditanase]

Description

Version search does not work if there are temporary parquet checkpoint files in the _delta_log folder.

The 2nd commit fixes a regression introduced in a recent commit that uses the kernel to look for max_version, which fails if the version doesn't start at 0 (rarely the case on an older table).

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.22%. Comparing base (2cc8081) to head (cd61fc5).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3549      +/-   ##
==========================================
- Coverage   74.23%   74.22%   -0.01%     
==========================================
  Files         150      150              
  Lines       44739    44742       +3     
  Branches    44739    44742       +3     
==========================================
- Hits        33213    33211       -2     
+ Misses       9384     9381       -3     
- Partials     2142     2150       +8     

☔ 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.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[ion-elgreco]

A test would be nice

[aditanase]

@ion-elgreco thanks! Added 1 tests to time_travel.rs, without it it crashes with:

called Result::unwrap() on an Err value: NotATable("../delta-rs/crates/test/tests/data/checkpoints_vacuumed/")

This is covering both changes I made, using the checkpoints_vacuumed test table

[aditanase]

@ion-elgreco FYI this seems to be the root cause of the issue introduced by migrating to kernel v11 for these operations:
https://github.com/delta-io/delta-kernel-rs/blob/main/kernel/src/log_segment.rs#L181

It does not handled compacted tables yet.