Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24273 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/24273 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- 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, 49 insertions(+), 28 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: I0e6bc981b4b91b4c92c6c29a3066b106ea722204 Gerrit-Change-Number: 24273 Gerrit-PatchSet: 5 Gerrit-Owner: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
