refactor(api)!: align signatures of Backend.table method; name is positional-only; everything else is required-keyword

Author: cpcloud

ibis/backends/__init__.py

@@ -1020,7 +1020,7 @@ def list_tables(
 
     @abc.abstractmethod
     def table(
-        self, name: str, database: tuple[str, str] | str | None = None
+        self, name: str, /, *, database: tuple[str, str] | str | None = None
     ) -> ir.Table:
         """Construct a table expression.
 

ibis/backends/athena/__init__.py

@@ -209,7 +209,7 @@ def create_table(
 
         return self.table(orig_table_ref.name, database=(catalog, db))
 
-    def table(self, name: str, database: str | None = None) -> ir.Table:
+    def table(self, name: str, /, *, database: str | None = None) -> ir.Table:
         """Construct a table expression.
 
         Parameters
@@ -223,7 +223,6 @@ def table(self, name: str, database: str | None = None) -> ir.Table:
         -------
         Table
             Table expression
-
         """
         table_loc = self._to_sqlglot_table(database)
 

ibis/backends/bigquery/__init__.py

@@ -582,6 +582,8 @@ def drop_database(
     def table(
         self,
         name: str,
+        /,
+        *,
         database: str | None = None,
     ) -> ir.Table:
         table_loc = self._to_sqlglot_table(database)

ibis/backends/databricks/__init__.py

@@ -199,7 +199,7 @@ def create_table(
 
         return self.table(name, database=(catalog, database))
 
-    def table(self, name: str, database: str | None = None) -> ir.Table:
+    def table(self, name: str, /, *, database: str | None = None) -> ir.Table:
         """Construct a table expression.
 
         Parameters

ibis/backends/duckdb/__init__.py

@@ -235,7 +235,7 @@ def create_table(
 
         return self.table(name, database=(catalog, database))
 
-    def table(self, name: str, database: str | None = None) -> ir.Table:
+    def table(self, name: str, /, *, database: str | None = None) -> ir.Table:
         """Construct a table expression.
 
         Parameters

ibis/backends/flink/__init__.py

@@ -247,6 +247,8 @@ def list_views(
     def table(
         self,
         name: str,
+        /,
+        *,
         database: str | None = None,
         catalog: str | None = None,
     ) -> ir.Table:

ibis/backends/polars/__init__.py

@@ -86,7 +86,7 @@ def version(self) -> str:
     def list_tables(self, like=None, database=None):
         return self._filter_with_like(list(self._tables.keys()), like)
 
-    def table(self, name: str, database: None = None) -> ir.Table:
+    def table(self, name: str, /, *, database: None = None) -> ir.Table:
         if database is not None:
             raise com.IbisError(
                 "Passing `database` to the Polars backend's `table()` method is not "

ibis/backends/sql/__init__.py

@@ -67,9 +67,7 @@ def _fetch_from_cursor(self, cursor, schema: sch.Schema) -> pd.DataFrame:
         return df
 
     def table(
-        self,
-        name: str,
-        database: tuple[str, str] | str | None = None,
+        self, name: str, /, *, database: tuple[str, str] | str | None = None
     ) -> ir.Table:
         """Construct a table expression.