Tamas Mate has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18305 )
Change subject: Revert "IMPALA-10737: Optimize the number of Iceberg API Metadata requests" ...................................................................... Revert "IMPALA-10737: Optimize the number of Iceberg API Metadata requests" This reverts commit cd10acdbb15c74eae21fa630bcb174104283560a. This commit has been reverted, because it blocks upgrading the Iceberg version to 0.13. In the newer Iceberg version the BaseTable serialization has been changed, it serializes the BaseTable to a SerializableTable sibiling class. This is a lightweigth Table class which does not have the necessary metadata that could be cached and reused by the ImpalaDs. SerializableTable utilization has to be further considered. Change-Id: I21e65cb3ab38d9e683223fb100d7ced90caa6edd Reviewed-on: http://gerrit.cloudera.org:8080/18305 Reviewed-by: Zoltan Borok-Nagy <borokna...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M common/thrift/CatalogObjects.thrift M common/thrift/CatalogService.thrift M fe/src/main/java/org/apache/impala/catalog/CatalogServiceCatalog.java M fe/src/main/java/org/apache/impala/catalog/FeIcebergTable.java M fe/src/main/java/org/apache/impala/catalog/IcebergTable.java D fe/src/main/java/org/apache/impala/catalog/IcebergTableLoadingException.java M fe/src/main/java/org/apache/impala/catalog/iceberg/IcebergCtasTarget.java M fe/src/main/java/org/apache/impala/catalog/local/CatalogdMetaProvider.java M fe/src/main/java/org/apache/impala/catalog/local/DirectMetaProvider.java M fe/src/main/java/org/apache/impala/catalog/local/LocalIcebergTable.java M fe/src/main/java/org/apache/impala/catalog/local/LocalTable.java M fe/src/main/java/org/apache/impala/catalog/local/MetaProvider.java M fe/src/main/java/org/apache/impala/planner/IcebergScanNode.java M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java M fe/src/main/java/org/apache/impala/service/Frontend.java M fe/src/main/java/org/apache/impala/util/IcebergUtil.java M fe/src/test/java/org/apache/impala/catalog/local/LocalCatalogTest.java M testdata/workloads/functional-query/queries/QueryTest/iceberg-insert.test 18 files changed, 182 insertions(+), 234 deletions(-) Approvals: Zoltan Borok-Nagy: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/18305 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I21e65cb3ab38d9e683223fb100d7ced90caa6edd Gerrit-Change-Number: 18305 Gerrit-PatchSet: 2 Gerrit-Owner: Tamas Mate <tma...@apache.org> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Tamas Mate <tma...@apache.org> Gerrit-Reviewer: Zoltan Borok-Nagy <borokna...@cloudera.com>