Skip to content

[SPARK-52805][INFRA] Add Java 17 build and install test pipeline#51499

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

[SPARK-52805][INFRA] Add Java 17 build and install test pipeline#51499
dongjoon-hyun wants to merge 1 commit intoapache:masterfrom
dongjoon-hyun:clean_build_java17

Conversation

@dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Jul 15, 2025

What changes were proposed in this pull request?

This PR aims to add Java 17 clean build test.

Why are the changes needed?

To clarify Java 17 clean build status always because Java 17 is the default Java version in the community.

Does this PR introduce any user-facing change?

No, this is an infra only change.

How was this patch tested?

Pass the CIs with newly added test pipeline.

https://github.com/dongjoon-hyun/spark/actions/runs/16299727217/job/46031048273

Screenshot 2025-07-15 at 10 54 26

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

No.

@github-actions github-actions bot added the INFRA label Jul 15, 2025
@dongjoon-hyun dongjoon-hyun changed the title SPARK-XXX: Add Java 17 clean build test [SPARK-52805][INFRA] Add Java 17 build and install test pipeline Jul 15, 2025
@dongjoon-hyun
Copy link
Member Author

Could you review this PR, @LuciferYang ?

Copy link
Contributor

@uros-db uros-db left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @dongjoon-hyun!

@dongjoon-hyun
Copy link
Member Author

It passed. Let me merge this first.

Screenshot 2025-07-15 at 10 54 26

@dongjoon-hyun dongjoon-hyun deleted the clean_build_java17 branch July 15, 2025 17:56
- uses: actions/setup-java@v4
with:
distribution: zulu
java-version: 17
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so Github Action will always install the latest java 17?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, I missed your comment here. Yes, it does. If we don't specify the exact version like 17.0.x, 17 is the alias.

In addition, actions/setup-java@v4 itself is an alias of v4.x.x, too. We are consuming the auto-updated status, @cloud-fan .

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.

4 participants

Comments