Skip to content

[SPARK-55386][INFRA] Run Java 17/25 Maven install tests on PR build only#54170

Closed
dongjoon-hyun wants to merge 1 commit intoapache:masterfrom
dongjoon-hyun:SPARK-55386
Closed

[SPARK-55386][INFRA] Run Java 17/25 Maven install tests on PR build only#54170
dongjoon-hyun wants to merge 1 commit intoapache:masterfrom
dongjoon-hyun:SPARK-55386

Conversation

@dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Feb 6, 2026

What changes were proposed in this pull request?

This PR aims to run Java 17 and Java 25 Maven build and install tests on PR builder only. In other words, it will run on a repository which is not apache/spark.

Since we still have daily Maven jobs, it's okay to skip this build and install tests.

Why are the changes needed?

To meet the ASF Policy:

1. 20 JOB RULE

All workflows MUST have a job concurrency level less than or equal to 20. This means a workflow cannot have more than 20 jobs running at the same time across all matrices.

Currently, our CI seems to trigger more than 20 jobs although it's not currently at the same time.

Screenshot 2026-02-05 at 20 49 58 Screenshot 2026-02-05 at 20 38 47

2. RUNNING TIME RULE

The average number of minutes a project uses per calendar week MUST NOT exceed the equivalent of 25 full-time runners (250,000 minutes, or 4,200 hours).

https://infra-reports.apache.org/#ghactions&project=spark&hours=168

This PR also reduces the CI cost by skipping Java 17 and 25 Maven build on commit builders.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Manual review.

Was this patch authored or co-authored using generative AI tooling?

No.

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

JIRA Issue Information

=== Sub-task SPARK-55386 ===
Summary: Run Java 17/25 Maven install tests on PR build only
Assignee: Dongjoon Hyun
Status: In Progress
Affected: ["4.2.0"]


This comment was automatically generated by GitHub Actions

@github-actions github-actions bot added the INFRA label Feb 6, 2026
@dongjoon-hyun
Copy link
Member Author

dongjoon-hyun commented Feb 6, 2026

The PR builder is running Java 17/25 Maven install tests currently.

Screenshot 2026-02-05 at 20 41 46

@dongjoon-hyun
Copy link
Member Author

Could you review this PR, @zhengruifeng ?

@dongjoon-hyun
Copy link
Member Author

Could you review this PR, @yaooqinn ?

@zhengruifeng
Copy link
Contributor

also cc @LuciferYang

@dongjoon-hyun
Copy link
Member Author

Thank you, @zhengruifeng and @LuciferYang !

Merged to master.

@dongjoon-hyun dongjoon-hyun deleted the SPARK-55386 branch February 6, 2026 06:02
@dongjoon-hyun
Copy link
Member Author

dongjoon-hyun commented Feb 6, 2026

For the record, the maven install tests are correctly skipped in the commit build.

Screenshot 2026-02-05 at 22 02 30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants