From a724dadb44b00e15045387a8062761646c7463cf Mon Sep 17 00:00:00 2001 From: Daniel Carl Jones Date: Sat, 6 Jun 2026 00:07:44 +0100 Subject: [PATCH] chore(refactor): rename SnapshotProducer::manifest_file to produce_manifest_file_list --- crates/iceberg/src/transaction/snapshot.rs | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/crates/iceberg/src/transaction/snapshot.rs b/crates/iceberg/src/transaction/snapshot.rs index 8e47226072..2e23b823c9 100644 --- a/crates/iceberg/src/transaction/snapshot.rs +++ b/crates/iceberg/src/transaction/snapshot.rs @@ -338,7 +338,10 @@ impl<'a> SnapshotProducer<'a> { writer.write_manifest_file().await } - async fn manifest_file( + /// Collects the list of manifest files to be included in the new snapshot. + /// + /// This method also writes the new manifests where required. + async fn produce_manifest_file_list( &mut self, snapshot_produce_operation: &OP, manifest_process: &MP, @@ -475,7 +478,7 @@ impl<'a> SnapshotProducer<'a> { })?; let new_manifests = self - .manifest_file(&snapshot_produce_operation, &process) + .produce_manifest_file_list(&snapshot_produce_operation, &process) .await?; manifest_list_writer.add_manifests(new_manifests.into_iter())?;