You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "abnobdoss (via GitHub)" <gi...@apache.org> on 2026/06/19 23:13:05 UTC

[PR] ci: cache Python bindings Rust artifacts [iceberg-rust]

abnobdoss opened a new pull request, #2679:
URL: https://github.com/apache/iceberg-rust/pull/2679

   ## Which issue does this PR close?
   
   - None. This is a CI maintenance change for the Python bindings test job.
   
   ## What changes are included in this PR?
   
   This adds a real Rust artifact cache to the Python bindings CI test job for host maturin builds on Windows and macOS.
   
   Concretely, this PR adds two steps before `PyO3/maturin-action`:
   
   - `Setup Rust toolchain`
   - `Cache Rust artifacts`
   
   Linux maturin builds may run inside the manylinux Docker container, where the host `target/` cache is not expected to help. Windows and macOS build on the host, so `swatinem/rust-cache` can restore Cargo artifacts for the expensive maturin compile.
   
   The cache uses a dedicated bindings key, and its `save-if` behavior matches the existing CI cache discipline: PRs may restore cache entries, but only `main` pushes create new ones.
   
   ## Are these changes tested?
   
     - Not run locally; this is a GitHub Actions workflow-only change.
   


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org

Re: [PR] ci: cache Python bindings Rust artifacts [iceberg-rust]

Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu commented on PR #2679:
URL: https://github.com/apache/iceberg-rust/pull/2679#issuecomment-4847421342

   thank you @abnobdoss 


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org

Re: [PR] ci: cache Python bindings Rust artifacts [iceberg-rust]

Posted by "kevinjqliu (via GitHub)" <gi...@apache.org>.
kevinjqliu merged PR #2679:
URL: https://github.com/apache/iceberg-rust/pull/2679


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org