Make replace_table_transaction @AbstractMethod on Catalog

Matches PR #498 (create_table_transaction) precedent. NoopCatalog gets
an explicit stub since it extends Catalog directly.

Author: smaheshwar-pltr

pyiceberg/catalog/__init__.py

@@ -479,12 +479,12 @@ def replace_table(
 
         Raises:
             NoSuchTableError: If the table does not exist.
-            TableAlreadyExistsError: If a view exists at the same identifier.
         """
         return self.replace_table_transaction(
             identifier, schema, location, partition_spec, sort_order, properties
         ).commit_transaction()
 
+    @abstractmethod
     def replace_table_transaction(
         self,
         identifier: str | Identifier,
@@ -515,9 +515,7 @@ def replace_table_transaction(
 
         Raises:
             NoSuchTableError: If the table does not exist.
-            TableAlreadyExistsError: If a view exists at the same identifier.
         """
-        raise NotImplementedError("replace_table_transaction is not supported for this catalog type")
 
     def _replace_staged_table(
         self,

pyiceberg/catalog/noop.py

@@ -28,6 +28,7 @@
 from pyiceberg.table import (
     CommitTableResponse,
     CreateTableTransaction,
+    ReplaceTableTransaction,
     Table,
 )
 from pyiceberg.table.sorting import UNSORTED_SORT_ORDER, SortOrder
@@ -68,6 +69,18 @@ def create_table_transaction(
     ) -> CreateTableTransaction:
         raise NotImplementedError
 
+    @override
+    def replace_table_transaction(
+        self,
+        identifier: str | Identifier,
+        schema: Schema | pa.Schema,
+        location: str | None = None,
+        partition_spec: PartitionSpec = UNPARTITIONED_PARTITION_SPEC,
+        sort_order: SortOrder = UNSORTED_SORT_ORDER,
+        properties: Properties = EMPTY_DICT,
+    ) -> ReplaceTableTransaction:
+        raise NotImplementedError
+
     @override
     def load_table(self, identifier: str | Identifier) -> Table:
         raise NotImplementedError