wangsheng has posted comments on this change. ( http://gerrit.cloudera.org:8080/17207 )
Change subject: IMPALA-10597: Enable setting Iceberg table properties ...................................................................... Patch Set 1: Hi Zoltan, this patch is ok, but I have a question: You mentioned that: other engines create Iceberg table may not contains these properties, so we need to allow setting these properties to make table usable. I understand that if we need to set table property by impala, catalog need to load table first. If Iceberg table do not contain these properties, does catalogd can load this kind of table normally? Maybe throw TableLoadingException in some situation? -- To view, visit http://gerrit.cloudera.org:8080/17207 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4b3506be4562a1ace3e6435867aadb3bdde7a8e2 Gerrit-Change-Number: 17207 Gerrit-PatchSet: 1 Gerrit-Owner: Zoltan Borok-Nagy <[email protected]> Gerrit-Reviewer: Gabor Kaszab <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: wangsheng <[email protected]> Gerrit-Comment-Date: Mon, 22 Mar 2021 01:59:49 +0000 Gerrit-HasComments: No
