[improvement](github-actions) Run BE UT (macOS) on master pushes only, once a day - #66958
Merged
Merged
Conversation
…, once a day The macOS build check runs on pull requests at ~100 builds/day against a 5-slot macOS concurrency limit, queueing up to 41.7h (48h sample: 1112h wall-clock for 496 runs, 355h spent queued). Run it on master pushes instead (~18 merges/day), keep one daily schedule for ccache warm-up, and add a 6h timeout to bound stuck runs. PR-level macOS compile coverage is intentionally dropped to relieve the queue; failures surface on master instead of on the PR. Co-Authored-By: Claude <noreply@anthropic.com>
Contributor
Author
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
Contributor
Author
|
skip buildall |
morningman
approved these changes
Aug 19, 2026
github-actions
Bot
added
the
approved
Contributor
|
PR approved by at least one committer and no changes requested. |
Contributor
|
PR approved by anyone and no changes requested. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Background
BE UT (macOS)is the largest consumer of the GitHub Actions queue: in a recent 48h sample it used 1112h of runner wall-clock (44% of all workflows, ~23 concurrent slots against the 5-slot macOS concurrency limit) and spent 355h queued, with a max queue wait of 41.7h. 496 runs were created (486 from PRs), of which 228 did no work (paths filter mismatch) and ~195 did a full BE build.This workflow only verifies that the BE compiles on macOS — it does not run unit tests (see the comment in the workflow). Running it on every PR push is far beyond what the macOS pool can absorb.
Changes
pushtomaster(~18 merges/day) instead of everypull_requestsynchronize (~100 builds/day).0 4,10,16,22 * * *→0 4 * * *(one daily cache warm-up instead of four).push/scheduletriggers left, the job-level paths filter has nothing to filter and is deleted; the build step now always runs.timeout-minutes: 360added: some runs were observed to hang for up to 24h.Behavior change
PRs no longer get a macOS compile check during development; failures surface on the master branch after merge. This trades PR-level macOS coverage for queue relief. If PR-level coverage is wanted again later, it can be re-added selectively (e.g. label-triggered or sampled) within the ~5-slot macOS capacity.
🤖 Generated with Claude Code