Update pyiceberg/table/update/validate.py

jayceslesar · Fokko · web-flow · commit f8c5fee45a9c · 2025-04-24T16:18:09.000-04:00
Co-authored-by: Fokko Driesprong &lt;fokko@apache.org&gt;
diff --git a/pyiceberg/table/update/validate.py b/pyiceberg/table/update/validate.py
@@ -55,8 +55,7 @@ def validation_history(
         if summary.operation not in matching_operations:
             continue
 
-        if snapshot not in snapshots:
-            snapshots.add(snapshot.snapshot_id)
+        snapshots.add(snapshot.snapshot_id)
         # TODO: Maybe do the IO in a separate thread at some point, and collect at the bottom (we can easily merge the sets 🤤 
         manifests_files.extend(
             [

Original file line number	Diff line number	Diff line change
`@@ -55,8 +55,7 @@ def validation_history(`
`55`	`55`	`if summary.operation not in matching_operations:`
`56`	`56`	`continue`
`57`	`57`
`58`		`- if snapshot not in snapshots:`
`59`		`- snapshots.add(snapshot.snapshot_id)`
	`58`	`+ snapshots.add(snapshot.snapshot_id)`
`60`	`59`	`# TODO: Maybe do the IO in a separate thread at some point, and collect at the bottom (we can easily merge the sets 🤤`
`61`	`60`	`manifests_files.extend(`
`62`	`61`	`[`