Noemi Pap-Takacs has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/22605 )
Change subject: IMPALA-13738 (Part1): Refactor IcebergTable.getPartialInfo() ...................................................................... IMPALA-13738 (Part1): Refactor IcebergTable.getPartialInfo() This patch aims to refactor Iceberg tables in the Catalog. It reduces the dependency on the internal hdfsTable_ object in IcebergTable.getPartialInfo(). Now we depend less on HdfsTable when we load an Iceberg table in local catalog mode. The remaining dependencies will be addressed in following patches. Now creating TPartialPartitionInfo is the responsibility of the partitions, not the tables: - FeFsPartition holds a basic default implementation for the unpartitioned tables, which includes all Iceberg tables (we handle Iceberg tables similarly to unpartitioned HDFS tables in the Catalog). - HdfsPartition extends this implementation with HDFS and Hive ACID specific fields. Testing: - existing Iceberg e2e tests in local catalog mode - existing Hive ACID tests in local catalog mode - ran test_iceberg_with_puffin.py in local catalog mode - LocalCatalog FE test Change-Id: I75d277fb555d0088a2e28111ff529a4605d734fa --- M fe/src/main/java/org/apache/impala/catalog/FeFsPartition.java M fe/src/main/java/org/apache/impala/catalog/FeFsTable.java M fe/src/main/java/org/apache/impala/catalog/HdfsPartition.java M fe/src/main/java/org/apache/impala/catalog/HdfsTable.java M fe/src/main/java/org/apache/impala/catalog/IcebergTable.java M fe/src/main/java/org/apache/impala/catalog/local/LocalFsTable.java 6 files changed, 194 insertions(+), 89 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/05/22605/5 -- To view, visit http://gerrit.cloudera.org:8080/22605 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I75d277fb555d0088a2e28111ff529a4605d734fa Gerrit-Change-Number: 22605 Gerrit-PatchSet: 5 Gerrit-Owner: Noemi Pap-Takacs <npaptak...@cloudera.com> Gerrit-Reviewer: Daniel Becker <daniel.bec...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Noemi Pap-Takacs <npaptak...@cloudera.com> Gerrit-Reviewer: Peter Rozsa <pro...@cloudera.com> Gerrit-Reviewer: Steve Carlin <scar...@cloudera.com> Gerrit-Reviewer: Zoltan Borok-Nagy <borokna...@cloudera.com>
