Noemi Pap-Takacs has uploaded this change for review. ( http://gerrit.cloudera.org:8080/19221
Change subject: IMPALA-11620: Enable setting 'write.format.default' ...................................................................... IMPALA-11620: Enable setting 'write.format.default' Enable setting 'write.format.default' to a different file format then what the table already contains. Before IMPALA-10610 we didn't support Iceberg tables with mixed-format data files. And we used 'write.format.default' to determine the file format of the table, which was only a temporary workaround. Because of this we didn't allow changing this table property if the table already contained different table formats. E.g. we didn't allow modifying 'write.format.default' to PARQUET if the table already contained ORC files, because it would have made the table unreadable for Impala. Since IMPALA-10610 'write.format.default' is not used to determine the Iceberg table's format anymore, so we can allow changing it. This tableproperty change is not synchronized between HMS and Iceberg metadata files in case of true external Hive Catalog tables. See IMPALA-11710. Testing: - E2E test in iceberg-alter.test Change-Id: I22d0a8a18fce99015fcfe1fd15cb4a4d4c2deaec --- M fe/src/main/java/org/apache/impala/analysis/AlterTableSetTblProperties.java M testdata/workloads/functional-query/queries/QueryTest/iceberg-alter.test 2 files changed, 3 insertions(+), 25 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/21/19221/3 -- To view, visit http://gerrit.cloudera.org:8080/19221 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I22d0a8a18fce99015fcfe1fd15cb4a4d4c2deaec Gerrit-Change-Number: 19221 Gerrit-PatchSet: 3 Gerrit-Owner: Noemi Pap-Takacs <npaptak...@cloudera.com> Gerrit-Reviewer: Noemi Pap-Takacs <npaptak...@cloudera.com> Gerrit-Reviewer: Tamas Mate <tma...@apache.org> Gerrit-Reviewer: Zoltan Borok-Nagy <borokna...@cloudera.com>