Skip to content

[DRAFT] Bump Hive to Version 4.0 #2998

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Conversation

cgivre
Copy link
Contributor

@cgivre cgivre commented Jul 9, 2025

DRILL-XXXX: Bump Hive to Version 4.0

Description

Drill used Hive libraries version 3.1. These libraries were not compatible with Java > 8, so all the unit tests were being skipped. This proposed PR updates the Hive libraries to version 4 which is compatible with Java 11.

Documentation

No user facing changes other than this allows Drill to be used with the latest version of Hive.

Testing

Ran existing unit tests.

@cgivre cgivre added code-cleanup refactoring PR related to code refactoring updates dependencies backport-to-stable This bug fix is applicable to the latest stable release and should be considered for inclusion there labels Jul 9, 2025
@cgivre
Copy link
Contributor Author

cgivre commented Jul 9, 2025

@shfshihuafeng I started messing with this and thought it might be a useful exercise to update Drill's Hive libraries to use Hive v4 so that we can run the unit tests. However.. it turned out to be more difficult than I imagined. Would you mind taking a look?

@shfshihuafeng
Copy link
Contributor

@shfshihuafeng I started messing with this and thought it might be a useful exercise to update Drill's Hive libraries to use Hive v4 so that we can run the unit tests. However.. it turned out to be more difficult than I imagined. Would you mind taking a look?

ok, i will do , it is necessary for unit test cases of DRILL-8526

@cgivre
Copy link
Contributor Author

cgivre commented Jul 10, 2025

@shfshihuafeng I started messing with this and thought it might be a useful exercise to update Drill's Hive libraries to use Hive v4 so that we can run the unit tests. However.. it turned out to be more difficult than I imagined. Would you mind taking a look?

ok, i will do , it is necessary for unit test cases of DRILL-8526

I'd like to do this, but not necessary for DRILL-8526. That said, I think I'm out of my league on this one.

Copy link
Contributor

@shfshihuafeng shfshihuafeng Jul 12, 2025

Choose a reason for hiding this comment

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

Bump Hive to 4.1.x, we got version info fromApache Hive

Hive 4.0.1 | Java 8
Hive 4.1.x | Java 17
Hive 4.2.x | Java 21

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-stable This bug fix is applicable to the latest stable release and should be considered for inclusion there code-cleanup dependencies refactoring PR related to code refactoring updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants