Hello Daniel Becker, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/22456
to look at the new patch set (#6).
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
---
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/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/IcebergPositionDeleteTable.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/IcebergContentFileStoreTest.java
13 files changed, 252 insertions(+), 174 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/56/22456/6
--
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: newpatchset
Gerrit-Change-Id: Id08baf4db32197bab74178777c4ba3fec98c2451
Gerrit-Change-Number: 22456
Gerrit-PatchSet: 6
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]>
