Hello Riza Suminto, Laszlo Gaal, Michael Smith, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/22705
to look at the new patch set (#12).
Change subject: IMPALA-13919: Allow running minicluster with Java 17
......................................................................
IMPALA-13919: Allow running minicluster with Java 17
IMPALA-11941 allowed building Impala and running tests with Java 17,
but it still uses Java 8 for minicluster components (e.g. Hadoop) and
skips several tests that would restart Hive. It should be possible to
use 17 for everything to be able to deprecate Java 8.
This patch mainly fixes Yarn+Hive+Tez startup issues with java 17 by
setting JAVA_TOOL_OPTIONS.
Another issues fixed is KuduHMSIntegrationTest: this test fails to
restart Kudu due to a bug in OpenJDK (see IMPALA-13856). The current
fix is to remove LD_PRELOAD to avoid loading libjsig (similarly to
the case when MINICLUSTER_JAVA_HOME is set). This works, but it
would be nice to clean up this area in a future patch.
Testing:
- ran exhaustive tests with Java 17
- ran core tests with default Java 8
Change-Id: If58b64a21d14a4a55b12dfe9ea0b9c3d5fe9c9cf
---
M bin/bootstrap_system.sh
M bin/create-test-configuration.sh
M bin/impala-config-java.sh
M bin/run-all-tests.sh
A bin/set-impala-java-tool-options.sh
M fe/pom.xml
M fe/src/test/java/org/apache/impala/customservice/KuduHMSIntegrationTest.java
M fe/src/test/java/org/apache/impala/testutil/TestUtils.java
M fe/src/test/resources/hive-site.xml.py
M testdata/bin/run-hive-server.sh
M testdata/bin/run-mini-dfs.sh
M tests/custom_cluster/test_kudu_table_create_without_hms.py
M tests/metadata/test_hms_integration.py
13 files changed, 146 insertions(+), 56 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/05/22705/12
--
To view, visit http://gerrit.cloudera.org:8080/22705
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If58b64a21d14a4a55b12dfe9ea0b9c3d5fe9c9cf
Gerrit-Change-Number: 22705
Gerrit-PatchSet: 12
Gerrit-Owner: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Laszlo Gaal <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
