Skip to content

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

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

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

Conversation

@kokila-19

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

Copy link
Copy Markdown
Contributor

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

@ayushtkn ayushtkn left a comment

Copy link
Copy Markdown
Member

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