use table snapshot_as_of_timestamp instead

pyiceberg/table/__init__.py

@@ -3841,7 +3841,7 @@ def metadata_log_entries(self) -> "pa.Table":
         ])
 
         def metadata_log_entry_to_row(metadata_entry: MetadataLogEntry) -> Dict[str, Any]:
-            latest_snapshot = self.tbl.metadata._snapshot_as_of_timestamp_ms(metadata_entry.timestamp_ms)
+            latest_snapshot = self.tbl.snapshot_as_of_timestamp(metadata_entry.timestamp_ms)
             return {
                 "timestamp": metadata_entry.timestamp_ms,
                 "file": metadata_entry.metadata_file,

pyiceberg/table/metadata.py

@@ -292,13 +292,6 @@ def snapshot_by_name(self, name: str) -> Optional[Snapshot]:
             return self.snapshot_by_id(ref.snapshot_id)
         return None
 
-    def _snapshot_as_of_timestamp_ms(self, timestamp_ms: int) -> Optional[Snapshot]:
-        """Return the snapshot that was current at the given timestamp or null if no such snapshot exists."""
-        for entry in reversed(self.snapshot_log):
-            if entry.timestamp_ms <= timestamp_ms:
-                return self.snapshot_by_id(entry.snapshot_id)
-        return None
-
     def current_snapshot(self) -> Optional[Snapshot]:
         """Get the current snapshot for this table, or None if there is no current snapshot."""
         if self.current_snapshot_id is not None: