Remove obsolete 'conn' and 'cursor' params from metastore methods (#1623)

Author: dreadatour

src/datachain/catalog/catalog.py

@@ -1004,11 +1004,9 @@ def update_dataset_version_with_warehouse_info(
 
         self.metastore.update_dataset_version(dataset, version, **values)
 
-    def update_dataset(
-        self, dataset: DatasetRecord, conn=None, **kwargs
-    ) -> DatasetRecord:
+    def update_dataset(self, dataset: DatasetRecord, **kwargs) -> DatasetRecord:
         """Updates dataset fields."""
-        dataset_updated = self.metastore.update_dataset(dataset, conn=conn, **kwargs)
+        dataset_updated = self.metastore.update_dataset(dataset, **kwargs)
         self.warehouse.rename_dataset_tables(dataset, dataset_updated)
         return dataset_updated
 

src/datachain/data_storage/db_engine.py

@@ -79,12 +79,7 @@ def compile_to_args(
         return result.string, params
 
     @abstractmethod
-    def execute(
-        self,
-        query,
-        cursor: Any | None = None,
-        conn: Any | None = None,
-    ) -> Iterator[tuple[Any, ...]]: ...
+    def execute(self, query) -> Iterator[tuple[Any, ...]]: ...
 
     def get_table(self, name: str) -> "Table":
         """Get a table by name, raising TableMissingError if not found."""
@@ -100,9 +95,7 @@ def get_table(self, name: str) -> "Table":
         return table
 
     @abstractmethod
-    def executemany(
-        self, query, params, cursor: Any | None = None
-    ) -> Iterator[tuple[Any, ...]]: ...
+    def executemany(self, query, params) -> None: ...
 
     @abstractmethod
     def execute_str(self, sql: str, parameters=None) -> Iterator[tuple[Any, ...]]: ...