Hello Quanlong Huang, ttttttz, Csaba Ringhofer, Michael Smith, Impala Public
Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/23916
to look at the new patch set (#5).
Change subject: IMPALA-13921: Make Java 17 the default in tests
......................................................................
IMPALA-13921: Make Java 17 the default in tests
This patch makes Java 17 the default in our builds. Once it has been
proven stable, we can explicitly remove support for Java 8 and
possibly Java 11, and remove all related code also.
Fixes required for this change:
* Fix DESCRIPTION output of bin/jenkins/build-all-flag-combinations.sh
* Use Java 17 in the docker containers
* Hive 3:
- Add javax.annotation-api as a direct dependency
- Disable DataNucleus (which is only needed for the interaction
between HMS and its backend DB)
- Remove Cleaner logic in the ORC EncodedReaderImpl, because its
API changed and it isn't needed for the Impala frontend/catalog.
* Hive 2:
- Patching Hive 2 and building with Java 17 is cumbersome. A cleaner
solution was to add some extra shimming around notification events
and catalog name.
- hive_metastore.thrift still need to be patched to add 'catName'
field to NotificationEvent, so we don't need to modify our C++ code
Generated-by: Github Copilot (Claude Sonnet 4.5)
Generated-by: Gemini Pro
Change-Id: I99cc3b752142e6936a039c02505e3c35e71277aa
---
M bin/impala-config-java.sh
M bin/jenkins/build-all-flag-combinations.sh
M docker/install_os_packages.sh
M
fe/src/compat-apache-hive-2/java/org/apache/impala/catalog/events/MetastoreEvents.java
M fe/src/main/java/org/apache/impala/catalog/events/EventExecutorService.java
M fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java
M
fe/src/main/java/org/apache/impala/catalog/events/MetastoreEventsProcessor.java
M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java
M java/ext-data-source/api/pom.xml
M testdata/bin/patch_hive.sh
M testdata/bin/patch_hive2.sh
A testdata/cluster/hive/java17/patch0-JDK-17-changes.diff
A testdata/cluster/hive2/patch0-CATALOG_NAME.diff
D testdata/cluster/hive2/patch0-HIVE-18755.diff
14 files changed, 166 insertions(+), 337 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/16/23916/5
--
To view, visit http://gerrit.cloudera.org:8080/23916
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I99cc3b752142e6936a039c02505e3c35e71277aa
Gerrit-Change-Number: 23916
Gerrit-PatchSet: 5
Gerrit-Owner: Zoltan Borok-Nagy <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Reviewer: ttttttz <[email protected]>
