Baunsgaard opened a new pull request, #16951:
URL: https://github.com/apache/iceberg/pull/16951

   ## Motivation
   
   ASF Infra flagged Iceberg as a top-5 consumer of the shared GitHub-hosted 
runner pool ([dev list 
thread](https://lists.apache.org/thread/36vxlql61gojbg639c86mnz78n57kvgm)). One 
of the optimizations raised there is JVM coverage: run a primary JVM for PRs 
and reserve the full JVM matrix for `main`, release branches, tags, and 
`full-ci`-labeled PRs. This PR implements exactly that JVM-axis slice for Spark 
CI, and nothing else.
   
   ## Change
   
   On pull requests (unless labeled `full-ci`), Java 21 runs only on the latest 
Spark (4.1 / Scala 2.13) as a canary. Java 17 still runs the full matrix. 
`main`, release branches, tags, and `full-ci` PRs keep the full Java 17 + Java 
21 matrix, so coverage at the integration points is unchanged.
   
   | Trigger | Spark jobs | Java 21 jobs |
   |---|---|---|
   | Pull request (default) | 10 | 2 (4.1/2.13 core + extensions) |
   | `main` / release / tag / `full-ci` PR | 16 | 8 |
   
   Implementation is two guarded `exclude` entries; the guard resolves to a 
non-matching value off PRs, so it excludes nothing on `main`.
   
   ## Scope
   
   This is intentionally the JVM-axis reduction only, and is complementary to:
   
   - #16566 (path/version-based PR matrix selection)
   - #16945 (single Java version for PR checks)
   
   It deliberately keeps a Java 21 canary on the latest Spark rather than 
dropping Java 21 from PRs entirely, to catch Java 21-specific regressions 
before merge at a cost of just 2 jobs.
   
   ## Notes
   
   - Uses a `full-ci` label as the manual override, consistent with the dev 
list proposal. The label must exist in the repo to be used.
     
     


-- 
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]

Reply via email to