gaogaotiantian opened a new pull request, #57660:
URL: https://github.com/apache/spark/pull/57660
### What changes were proposed in this pull request?
Remove the `github.ref != 'refs/heads/branch-4.x'` clause from the
`call-build-and-test` job condition in `build_main.yml` on `master`. For a
`push` event, GitHub Actions always evaluates the workflow file from the ref
being pushed, so `master`'s copy of this file is only ever evaluated for pushes
to `master` -- meaning that comparison can never be false and is dead.
Disabling post-merge CI on `branch-4.x` is instead handled in that branch's own
copy of the workflow (separate PR against `branch-4.x`).
The fork sync-`master` skip is retained:
```yaml
if: >-
github.repository == 'apache/spark'
|| github.ref != 'refs/heads/master'
```
### Why are the changes needed?
The other build workflows (e.g. `build_java21.yml`, `build_python_3.11.yml`)
are already managed per-branch, diverging in name/schedule/`branch:` between
`master` and `branch-4.x`. `build_main.yml` was the exception, encoding
branch-specific behavior via a runtime `github.ref` check that is inert on
`master`. Aligning it with the rest keeps each branch's workflow file
self-describing.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
N/A. CI-configuration-only change; behavior on `master` is unchanged.
### Was this patch authored or co-authored using generative AI tooling?
Generated-by: Claude Code (Opus 4.8)
This pull request and its description were written by Isaac.
--
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]
