ethan-tyler opened a new pull request, #1995:
URL: https://github.com/apache/iceberg-rust/pull/1995

   ## Summary
   
   This PR validates DataFusion 52.0.0 (pre-release @ `9a9ff8d`) compatibility 
with iceberg-rust for the upcoming DataFusion release.
   
   **Reference**: apache/datafusion#18566
   
   ## Changes
   
   ### Dependency Updates
   - DataFusion: `51.0` → git ref `9a9ff8d` (pre-release 52)
   - Arrow/Parquet: `57.0` → `57.1`
   
   ### Docker Networking Fix (macOS compatibility)
   Fixed pre-existing issue where Docker-based tests failed on macOS:
   - Use `localhost` with port mappings instead of container IPs (172.x.x.x not 
routable from macOS host with Docker Desktop)
   - Added missing port mappings for MinIO and GCS emulator
   
   ## Test Results
   
   All tests passing locally:
   
   | Test Suite | Tests | Result |
   |------------|-------|--------|
   | iceberg-datafusion unit | 49 | ✅ |
   | iceberg-datafusion integration | 9 | ✅ |
   | iceberg core | 1054 | ✅ |
   | sqllogictests | 11 | ✅ |
   | Docker integration (REST + Spark) | 7 | ✅ |
   | GCS file_io | 3 | ✅ |
   | S3 file_io | 8 | ✅ |
   
   **Environment:**
   - Rust: 1.89.0-nightly (MSRV 1.88)
   - OS: macOS (aarch64)
   
   ## Notes
   
   - No code changes required for DataFusion 52 API compatibility
   - Arrow 57.1 alignment verified
   - Docker networking fix is a separate improvement that enables macOS testing
   - This is a WIP/validation PR - ready to finalize once DataFusion 52.0.0 is 
released on crates.io


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to