rename optimize -> maintenance

Author: jayceslesar

pyiceberg/table/__init__.py

@@ -80,14 +80,14 @@
 from pyiceberg.schema import Schema
 from pyiceberg.table.inspect import InspectTable
 from pyiceberg.table.locations import LocationProvider, load_location_provider
+from pyiceberg.table.maintenance import MaintenanceTable
 from pyiceberg.table.metadata import (
     INITIAL_SEQUENCE_NUMBER,
     TableMetadata,
 )
 from pyiceberg.table.name_mapping import (
     NameMapping,
 )
-from pyiceberg.table.optimize import OptimizeTable
 from pyiceberg.table.refs import SnapshotRef
 from pyiceberg.table.snapshots import (
     Snapshot,
@@ -1025,13 +1025,13 @@ def inspect(self) -> InspectTable:
         return InspectTable(self)
 
     @property
-    def optimize(self) -> OptimizeTable:
-        """Return the OptimizeTable object to optimize.
+    def maintenance(self) -> MaintenanceTable:
+        """Return the MaintenanceTable object for maintenance.
 
         Returns:
-            OptimizeTable object based on this Table.
+            MaintenanceTable object based on this Table.
         """
-        return OptimizeTable(self)
+        return MaintenanceTable(self)
 
     def refresh(self) -> Table:
         """Refresh the current table metadata.

pyiceberg/table/optimize.py pyiceberg/table/maintenance.pypyiceberg/table/optimize.py renamed to pyiceberg/table/maintenance.py

@@ -31,7 +31,7 @@
     from pyiceberg.table import Table
 
 
-class OptimizeTable:
+class MaintenanceTable:
     tbl: Table
 
     def __init__(self, tbl: Table) -> None:

tests/table/test_remove_orphans.py

@@ -69,17 +69,17 @@ def test_remove_orphaned_files(catalog: Catalog) -> None:
     assert orphaned_file.exists()
 
     # assert no files deleted if dry run...
-    tbl.optimize.remove_orphaned_files(dry_run=True)
+    tbl.maintenance.remove_orphaned_files(dry_run=True)
     assert orphaned_file.exists()
 
     # should not delete because it was just created...
-    tbl.optimize.remove_orphaned_files()
+    tbl.maintenance.remove_orphaned_files()
     assert orphaned_file.exists()
 
     # modify creation date to be older than 3 days
     five_days_ago = (datetime.now() - timedelta(days=5)).timestamp()
     os.utime(orphaned_file, (five_days_ago, five_days_ago))
-    tbl.optimize.remove_orphaned_files()
+    tbl.maintenance.remove_orphaned_files()
     assert not orphaned_file.exists()
 
     # assert that all known files still exist...
@@ -118,7 +118,7 @@ def test_remove_orphaned_files_with_invalid_file_doesnt_error(catalog: Catalog)
     tbl.append(df)
 
     file_that_does_not_exist = "foo/bar.baz"
-    with patch.object(type(tbl.optimize), "orphaned_files", return_value={file_that_does_not_exist}):
+    with patch.object(type(tbl.maintenance), "orphaned_files", return_value={file_that_does_not_exist}):
         with patch.object(tbl.io, "delete", wraps=tbl.io.delete) as mock_delete:
-            tbl.optimize.remove_orphaned_files(timedelta(days=3))
+            tbl.maintenance.remove_orphaned_files(timedelta(days=3))
             mock_delete.assert_called_with(file_that_does_not_exist)