[ 
https://issues.apache.org/jira/browse/IMPALA-14631?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18045508#comment-18045508
 ] 

Noémi Pap-Takács commented on IMPALA-14631:
-------------------------------------------

Commit 0b3bc412f2b09f4818cb0c50fccec5cfb3ddfe8d in impala's branch 
refs/heads/master from Noemi Pap-Takacs
[ [https://gitbox.apache.org/repos/asf?p=impala.git;h=0b3bc412f] ]

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 <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>

> Refactor IcebergTable.getPartialInfo()
> --------------------------------------
>
>                 Key: IMPALA-14631
>                 URL: https://issues.apache.org/jira/browse/IMPALA-14631
>             Project: IMPALA
>          Issue Type: Sub-task
>          Components: Catalog
>            Reporter: Noémi Pap-Takács
>            Assignee: Noémi Pap-Takács
>            Priority: Major
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to