Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20143 )
Change subject: IMPALA-11941: (Addendum) ease testing other JDKs ...................................................................... IMPALA-11941: (Addendum) ease testing other JDKs Makes it simpler to build with one JDK and run tests with another. TEST_JDK_VERSION sets IMPALA_JDK_VERSION before running tests, so the Impala cluster is started with that JDK. TEST_JAVA_HOME_OVERRIDE sets IMPALA_JAVA_HOME_OVERRIDE if a non-OS version of Java is required. Restart Kudu with original JAVA_HOME in frontend tests. Also skips restarting Hive, Kudu, and Ranger in tests as they'll restart with a different JDK than originally started with. Testing: 1. built normally 2. ran "TEST_JDK_VERSION=17 run-all-tests.sh" 3. verified various logs contain "java.specification.version:17" Change-Id: I46b5515efd9537d63b843dbc42aa93b376efce00 Reviewed-on: http://gerrit.cloudera.org:8080/20143 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M bin/run-all-tests.sh M fe/src/test/java/org/apache/impala/customservice/KuduHMSIntegrationTest.java M tests/authorization/test_ranger.py M tests/common/environ.py M tests/common/skip.py M tests/custom_cluster/test_catalog_hms_failures.py M tests/custom_cluster/test_custom_hive_configs.py M tests/custom_cluster/test_kudu.py M tests/custom_cluster/test_kudu_table_create_without_hms.py M tests/custom_cluster/test_metastore_events_cleanup.py 10 files changed, 40 insertions(+), 5 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/20143 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I46b5515efd9537d63b843dbc42aa93b376efce00 Gerrit-Change-Number: 20143 Gerrit-PatchSet: 11 Gerrit-Owner: Michael Smith <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Laszlo Gaal <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]>
