Skip to content

HIVE-29575: Upgrade GitHub Actions, setup-java and checkout#6445

Merged
ayushtkn merged 1 commit intoapache:masterfrom
kokila-19:test
Apr 23, 2026
Merged

HIVE-29575: Upgrade GitHub Actions, setup-java and checkout#6445
ayushtkn merged 1 commit intoapache:masterfrom
kokila-19:test

Conversation

@kokila-19
Copy link
Copy Markdown
Contributor

@kokila-19 kokila-19 commented Apr 22, 2026

What changes were proposed in this pull request?
This PR updates GitHub Actions dependencies used in CI, specifically upgrading actions/setup-java to v5 and actions/checkout to v3.

Why are the changes needed?
CI builds were failing due to incorrect or missing JAVA_HOME configuration when using the older version of actions/setup-java. Upgrading actions/setup-java resolves this.

Does this PR introduce any user-facing change?
No

How was this patch tested?
CI pipeline run shows successful build after upgrading actions/setup-java

@sonarqubecloud
Copy link
Copy Markdown

@kokila-19
Copy link
Copy Markdown
Contributor Author

We have one test failure
PR: https://ci.hive.apache.org/blue/organizations/jenkins/hive-precommit/detail/PR-6445/3/tests/.
Master branch: https://ci.hive.apache.org/blue/organizations/jenkins/hive-precommit/detail/master/2907/tests

Its reproducible on master branch (without this change). It appears to be related to #6374 , as the failure goes away when reverting it. And this will be resolved with #6446

Copy link
Copy Markdown
Member

@ayushtkn ayushtkn left a comment

Choose a reason for hiding this comment

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

LGTM
There is a running known failure in the CI.
We had a green build with the fix: https://ci.hive.apache.org/blue/organizations/jenkins/hive-precommit/detail/PR-6445/4/pipeline

Considering this fixes the Github Actions & it is failing everywhere, not waiting for the tests to end. Merging to unblock other PRs

@ayushtkn ayushtkn merged commit 7b05183 into apache:master Apr 23, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants