mirror of
https://github.com/apache/impala.git
synced 2025-12-19 09:58:28 -05:00
6f2d9a24d8
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 Reviewed-on: http://gerrit.cloudera.org:8080/22705 Tested-by: Impala Public Jenkins <impala-public-jenkins@cloudera.com> Reviewed-by: Riza Suminto <riza.suminto@cloudera.com> Reviewed-by: Michael Smith <michael.smith@cloudera.com>
43 lines
1.9 KiB
Python
43 lines
1.9 KiB
Python
# Licensed to the Apache Software Foundation (ASF) under one
|
|
# or more contributor license agreements. See the NOTICE file
|
|
# distributed with this work for additional information
|
|
# regarding copyright ownership. The ASF licenses this file
|
|
# to you under the Apache License, Version 2.0 (the
|
|
# "License"); you may not use this file except in compliance
|
|
# with the License. You may obtain a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing,
|
|
# software distributed under the License is distributed on an
|
|
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
|
|
# KIND, either express or implied. See the License for the
|
|
# specific language governing permissions and limitations
|
|
# under the License.
|
|
|
|
from __future__ import absolute_import, division, print_function
|
|
import pytest
|
|
import os
|
|
|
|
from tests.common.custom_cluster_test_suite import CustomClusterTestSuite
|
|
from tests.common.skip import SkipIfHive3, SkipIf
|
|
|
|
IMPALA_HOME = os.getenv('IMPALA_HOME')
|
|
HIVE_SITE_WITHOUT_HMS_DIR = IMPALA_HOME + '/fe/src/test/resources/hive-site-without-hms'
|
|
TBL_NAME = "test_kudu_table_create_without_hms"
|
|
|
|
|
|
class TestCreatingKuduTableWithoutHMS(CustomClusterTestSuite):
|
|
"""Test creating kudu managed table without hms"""
|
|
|
|
@SkipIfHive3.without_hms_not_supported # TODO: will we ever support this?
|
|
@SkipIf.is_test_jdk
|
|
@pytest.mark.execute_serially
|
|
@CustomClusterTestSuite.with_args(hive_conf_dir=HIVE_SITE_WITHOUT_HMS_DIR)
|
|
def test_kudu_table_create_without_hms(self, unique_database):
|
|
table_name = unique_database + "." + TBL_NAME
|
|
self.execute_query_expect_success(self.client, "create table %s "
|
|
"(id int, primary key(id)) stored as kudu" % table_name)
|
|
self.execute_query_expect_success(self.client, "select * from %s" % table_name)
|
|
self.execute_query("drop table if exists %s" % table_name)
|