Csaba Ringhofer has uploaded this change for review. ( 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"). 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, 56 insertions(+), 26 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/73/24273/1 -- 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: newchange Gerrit-Change-Id: I0e6bc981b4b91b4c92c6c29a3066b106ea722204 Gerrit-Change-Number: 24273 Gerrit-PatchSet: 1 Gerrit-Owner: Csaba Ringhofer <[email protected]>
