Add snapshot-log pruning to ExpireSnapshots operation

arnaudbriche · arnaudbriche · commit 011df8d0fbc2 · 2026-01-13T11:12:14.000+01:00
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);`