Noemi Pap-Takacs has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/19387 )
Change subject: IMPALA-11807: Fix TestIcebergTable.test_avro_file_format and test_mixed_file_format ...................................................................... IMPALA-11807: Fix TestIcebergTable.test_avro_file_format and test_mixed_file_format Iceberg hardcodes URIs in metadata files. If the table was written in a certain storage location and then moved to another file system, the hardcoded URIs will still point to the old location instead of the current one. Therefore Impala will be unable to read the table. TestIcebergTable.test_avro_file_format and test_mixed_file_format use Hive from Impala to write tables. If the tables are created in a different file system than the one they will be read from, the tests fail due to the invalid URIs. Skipping these 2 tests if testing is not done on HDFS. Updated the data load schema of the 2 test tables created by Hive and set LOCATION to the same as in the previous test tables. If this makes it possible to rewrite the URIs in the metadata and makes the tables accessible from another file system as well later, then the tests can be enabled again. Testing: - Testing locally on HDFS minicluster - Triggered an Ozone build to verify that it is skipped on a different file system Change-Id: Ie2f126de80c6e7f825d02f6814fcf69ae320a781 --- M testdata/datasets/functional/functional_schema_template.sql M tests/common/skip.py M tests/query_test/test_iceberg.py 3 files changed, 7 insertions(+), 3 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/87/19387/4 -- To view, visit http://gerrit.cloudera.org:8080/19387 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ie2f126de80c6e7f825d02f6814fcf69ae320a781 Gerrit-Change-Number: 19387 Gerrit-PatchSet: 4 Gerrit-Owner: Noemi Pap-Takacs <npaptak...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com> Gerrit-Reviewer: Noemi Pap-Takacs <npaptak...@cloudera.com> Gerrit-Reviewer: Tamas Mate <tma...@apache.org>
