Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 8 additions & 5 deletions AGENTS.md
Original file line number Diff line number Diff line change
Expand Up @@ -205,8 +205,11 @@ compile — they have their own visitor implementors:
proto message usually needs: POJO + visitor wiring + both proto converters + a
round-trip test, and often `ExpressionCreator` factories and `dsl/SubstraitBuilder`
helpers for ergonomics.
- When monitoring PR checks, budget for a long tail: the **macOS `Build Isthmus Native
Image`** job is the long pole — it `needs:` the `java` + `integration` jobs (so it
starts late), then AOT-compiles for ~15–20 min on a slower macOS runner. A PR staying
yellow after the other checks pass usually just means that job is still running, not
that it's stuck or failing.
- When monitoring PR checks: the per-PR native-image job (`Build Isthmus Native Image` in
`pr.yml`) builds **Linux only**, uses `--quick-build-native`, and runs **in parallel**
with the `java` + `integration` jobs (no `needs:`), so it's fast and not a long pole. The
**macOS** native image is *not* built on PRs — it runs out-of-band in
`.github/workflows/native-image-macos.yml` (push to `main`, a weekly backstop, and
`workflow_dispatch`), fully optimized to mirror `release.yml`. So a PR that stays yellow
isn't waiting on a macOS native build, and macOS-specific native regressions surface on
`main` or the weekly run rather than on the PR.
Loading