Merge pull request JanKaul#300 from arnaudbriche/prune-snapshot-log-during-exire-snapshots

JanKaul · web-flow · commit 31d6d9116e95 · 2026-01-13T15:00:13.000+01:00
Add snapshot-log pruning to ExpireSnapshots operation
diff --git a/iceberg-rust/src/catalog/commit.rs b/iceberg-rust/src/catalog/commit.rs
@@ -513,6 +513,10 @@ pub fn apply_table_updates(
                 for id in snapshot_ids {
                     metadata.snapshots.remove(&id);
                 }
+
+                metadata
+                    .snapshot_log
+                    .retain(|e| metadata.snapshots.contains_key(&e.snapshot_id));
             }
             TableUpdate::RemoveSnapshotRef { ref_name } => {
                 metadata.refs.remove(&ref_name);

Original file line number	Diff line number	Diff line change
`@@ -513,6 +513,10 @@ pub fn apply_table_updates(`
`513`	`513`	`for id in snapshot_ids {`
`514`	`514`	`metadata.snapshots.remove(&id);`
`515`	`515`	`}`
	`516`	`+`
	`517`	`+ metadata`
	`518`	`+ .snapshot_log`
	`519`	`+ .retain(\|e\| metadata.snapshots.contains_key(&e.snapshot_id));`
`516`	`520`	`}`
`517`	`521`	`TableUpdate::RemoveSnapshotRef { ref_name } => {`
`518`	`522`	`metadata.refs.remove(&ref_name);`