Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22456 )
Change subject: IMPALA-13739: Part2: Introduce IcebergFileDescriptor ...................................................................... IMPALA-13739: Part2: Introduce IcebergFileDescriptor FileDescriptor is currently used by both HdfsTable and IcebergTable. It has two flatbuffers internally, however, one of them is only relevant for Iceberg tables. With this change we move the Iceberg part of FileDescriptor into a separate class called IcebergFileDescriptor. Change-Id: Id08baf4db32197bab74178777c4ba3fec98c2451 Reviewed-on: http://gerrit.cloudera.org:8080/22456 Reviewed-by: Daniel Becker <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/analysis/OptimizeStmt.java M fe/src/main/java/org/apache/impala/analysis/TableRef.java M fe/src/main/java/org/apache/impala/catalog/FeIcebergTable.java M fe/src/main/java/org/apache/impala/catalog/FileDescriptor.java M fe/src/main/java/org/apache/impala/catalog/FileMetadataLoader.java M fe/src/main/java/org/apache/impala/catalog/IcebergContentFileStore.java M fe/src/main/java/org/apache/impala/catalog/IcebergDeleteTable.java M fe/src/main/java/org/apache/impala/catalog/IcebergEqualityDeleteTable.java A fe/src/main/java/org/apache/impala/catalog/IcebergFileDescriptor.java M fe/src/main/java/org/apache/impala/catalog/IcebergFileMetadataLoader.java M fe/src/main/java/org/apache/impala/catalog/IcebergPositionDeleteTable.java M fe/src/main/java/org/apache/impala/catalog/IcebergTable.java M fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java M fe/src/main/java/org/apache/impala/planner/IcebergScanNode.java M fe/src/main/java/org/apache/impala/planner/IcebergScanPlanner.java M fe/src/test/java/org/apache/impala/catalog/FileMetadataLoaderTest.java M fe/src/test/java/org/apache/impala/catalog/IcebergContentFileStoreTest.java 17 files changed, 301 insertions(+), 211 deletions(-) Approvals: Daniel Becker: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/22456 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id08baf4db32197bab74178777c4ba3fec98c2451 Gerrit-Change-Number: 22456 Gerrit-PatchSet: 11 Gerrit-Owner: Gabor Kaszab <[email protected]> Gerrit-Reviewer: Daniel Becker <[email protected]> Gerrit-Reviewer: Gabor Kaszab <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
