fix(core): handle Result type after get_actions sync conversion

[yousefsaad12]

Description

Fixed compilation errors that occurred after get_actions was converted from async to sync as part of #3837. The refactoring changed the function signature but left several call sites using the old async pattern, resulting in type mismatches and compilation failures across multiple modules.

Changes Made

• Added ? operator to properly handle Result<Vec<Action>, DeltaTableError> returned by get_actions()
• Removed .await calls since get_actions is now synchronous and no longer returns a Future
• Fixed pattern matching from &&Action to Action in filter predicates to match the updated return type structure

Files Modified

• crates/core/src/kernel/transaction/conflict_checker.rs - Added ? operator and fixed iterator pattern matching
• crates/core/src/operations/load_cdf.rs - Added ? operator to handle Result type
• crates/core/src/operations/merge/mod.rs - Removed .await calls (3 locations)
• crates/core/src/operations/write/mod.rs - Fixed pattern matching in filter predicates (3 locations)

Testing

All compilation errors have been resolved. Testing results:

• 114 core library unit tests passing
• Code formatted with cargo fmt
• Linted with cargo clippy with no new warnings
• Verified no regression in existing functionality

Impact

This change addresses compilation failures without altering runtime behavior or public API surface. The modifications are minimal and focused solely on adapting call sites to the new synchronous get_actions function signature.

Related Issue(s)

• Relates to Make get_actions() sync #3837
• Fixes compilation errors following the get_actions async-to-sync conversion

Documentation

No documentation changes are required as this is an internal implementation change that maintains backward compatibility in functionality while adapting to the new synchronous API.

[github-actions]

ACTION NEEDED

delta-rs follows the Conventional Commits specification for release automation.

The PR title and description are used as the merge commit message. Please update your PR title and description to match the specification.

[codecov]

Codecov Report

❌ Patch coverage is 46.15385% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.76%. Comparing base (ad91db7) to head (8893f00).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
crates/core/src/operations/load_cdf.rs	25.00%	0 Missing and 3 partials ⚠️
crates/core/src/operations/write/mod.rs	0.00%	0 Missing and 3 partials ⚠️
...es/core/src/kernel/transaction/conflict_checker.rs	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3846      +/-   ##
==========================================
- Coverage   73.78%   73.76%   -0.03%     
==========================================
  Files         152      152              
  Lines       39144    39116      -28     
  Branches    39144    39116      -28     
==========================================
- Hits        28883    28853      -30     
- Misses       8987     8989       +2     
  Partials     1274     1274              

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