1. support returning rewrite results

2. write tests for rewrite manifests

Author: amitgilad3

pyiceberg/table/__init__.py

@@ -112,7 +112,7 @@
     update_table_metadata,
 )
 from pyiceberg.table.update.schema import UpdateSchema
-from pyiceberg.table.update.snapshot import ManageSnapshots, UpdateSnapshot, _FastAppendFiles
+from pyiceberg.table.update.snapshot import ManageSnapshots, RewriteManifestsResult, UpdateSnapshot, _FastAppendFiles
 from pyiceberg.table.update.spec import UpdateSpec
 from pyiceberg.table.update.statistics import UpdateStatistics
 from pyiceberg.transforms import IdentityTransform
@@ -420,9 +420,12 @@ def update_snapshot(self, snapshot_properties: Dict[str, str] = EMPTY_DICT) -> U
         """
         return UpdateSnapshot(self, io=self._table.io, snapshot_properties=snapshot_properties)
 
-    def rewrite_manifests(self, spec_id: Optional[int] = None) -> None:
+    def rewrite_manifests(self, spec_id: Optional[int] = None) -> RewriteManifestsResult:
+        if self._table.current_snapshot() is None:
+            return RewriteManifestsResult(rewritten_manifests=[], added_manifests=[])
         with self.update_snapshot().rewrite() as rewrite:
-            rewrite.rewrite_manifests()
+            rewritten = rewrite.rewrite_manifests()
+            return rewritten
 
     def append(self, df: pa.Table, snapshot_properties: Dict[str, str] = EMPTY_DICT) -> None:
         """
@@ -1174,7 +1177,7 @@ def add_files(
     def rewrite_manifests(
         self,
         spec_id: Optional[int] = None,
-    ) -> None:
+    ) -> RewriteManifestsResult:
         """
         Shorthand API for Rewriting manifests for the table.
 
@@ -1183,7 +1186,7 @@ def rewrite_manifests(
 
         """
         with self.transaction() as tx:
-            tx.rewrite_manifests(spec_id=spec_id)
+            return tx.rewrite_manifests(spec_id=spec_id)
 
     def update_spec(self, case_sensitive: bool = True) -> UpdateSpec:
         return UpdateSpec(Transaction(self, autocommit=True), case_sensitive=case_sensitive)