Impala Public Jenkins has submitted this change and it was merged. ( 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 Reviewed-on: http://gerrit.cloudera.org:8080/22605 Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- 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(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: I75d277fb555d0088a2e28111ff529a4605d734fa Gerrit-Change-Number: 22605 Gerrit-PatchSet: 9 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>
