zhengruifeng opened a new pull request, #56342:
URL: https://github.com/apache/spark/pull/56342
### What changes were proposed in this pull request?
Add `${{ runner.os }}-` as a prefix to every cache `key:` and
`restore-keys:` value in all CI workflow files that did not already have it:
- `build_and_test.yml`: `build-`, `docs-maven-`, `tpcds-`
- `build_sparkr_window.yml`: `build-sparkr-windows-maven-`
- `maven_test.yml`: `build-`, `java${{...}}-maven-`
- `build_python_connect.yml` / `build_python_connect40.yml`:
`build-spark-connect-python-only-`, `coursier-build-spark-connect-python-only-`
- `python_hosted_runner_test.yml`: `build-`
- `publish_snapshot.yml`: `snapshot-maven-`
Keys that already carried the `${{ runner.os }}-` prefix (e.g. `${{
runner.os }}-coursier-`) are unchanged.
### Why are the changes needed?
Without an OS prefix, cache entries from Linux and Windows runners share the
same key namespace. A cache written by a Linux runner can be restored on a
Windows runner (and vice-versa), causing subtle build failures or
stale-artifact issues. Prefixing with `${{ runner.os }}` ensures each OS has
its own isolated cache.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
CI workflow change only; no code logic changed. The cache keys are verified
by inspection.
### Was this patch authored or co-authored using generative AI tooling?
Generated-by: Claude Sonnet 4.6
--
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]
