Hello Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24273
to look at the new patch set (#2).
Change subject: IMPALA-14804: (addendum) Refactor
IcebergTable.loadFileMetadata()
......................................................................
IMPALA-14804: (addendum) Refactor IcebergTable.loadFileMetadata()
Extract file descriptor loading from IcebergTable.loadFileMetadata()
to static function IcebergFileMetadataLoader.loadContentFilesStore().
The main goal is to make this reusable from
https://gerrit.cloudera.org/#/c/24049/ and avoid making that patch
more noisy.
Also has a diagnostic change: "Loaded Iceberg file descriptors"
event in catalog timeline is moved after IcebergContentFileStore
creation. It makes much more sense to track the time there
instead of the previous state ("Loaded avro schema").
Generated-by: Claude Sonnet 4.6
Change-Id: I0e6bc981b4b91b4c92c6c29a3066b106ea722204
---
M fe/src/main/java/org/apache/impala/catalog/IcebergFileMetadataLoader.java
M fe/src/main/java/org/apache/impala/catalog/IcebergTable.java
2 files changed, 55 insertions(+), 26 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/73/24273/2
--
To view, visit http://gerrit.cloudera.org:8080/24273
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I0e6bc981b4b91b4c92c6c29a3066b106ea722204
Gerrit-Change-Number: 24273
Gerrit-PatchSet: 2
Gerrit-Owner: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
