hililiwei commented on code in PR #533:
URL: https://github.com/apache/iceberg-python/pull/533#discussion_r1530185804
##########
pyiceberg/table/__init__.py:
##########
@@ -3014,3 +3145,35 @@ def _new_field_id(self) -> int:
def _is_duplicate_partition(self, transform: Transform[Any, Any],
partition_field: PartitionField) -> bool:
return partition_field.field_id not in self._deletes and
partition_field.transform == transform
+
+
+def ancestors_between(to_snapshot: int, from_snapshot: Optional[int], table:
Table) -> Iterable[Snapshot]:
Review Comment:
Should these methods be placed here? Is there a more suitable place for them?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
