refactor: remove unused _wrap method and improve deprecation message formatting in TableProvider

Author: kosiew

python/datafusion/table_provider.py

@@ -18,8 +18,8 @@
 
 from __future__ import annotations
 
-from typing import Any
 import warnings
+from typing import Any
 
 import datafusion._internal as df_internal
 
@@ -42,15 +42,6 @@ def __init__(self, table_provider: _InternalTableProvider) -> None:
 
         self._table_provider = table_provider
 
-    # ------------------------------------------------------------------
-    # constructors
-    # ------------------------------------------------------------------
-    @classmethod
-    def _wrap(cls, provider: _InternalTableProvider | TableProvider) -> TableProvider:
-        if isinstance(provider, cls):
-            return provider
-        return cls(provider)
-
     @classmethod
     def from_capsule(cls, capsule: Any) -> TableProvider:
         """Create a :class:`TableProvider` from a PyCapsule."""
@@ -70,7 +61,11 @@ def from_dataframe(cls, df: Any) -> TableProvider:
 
     @classmethod
     def from_view(cls, df: Any) -> TableProvider:
-        """Deprecated. Use :meth:`DataFrame.into_view` or :meth:`TableProvider.from_dataframe`."""
+        """Deprecated.
+
+        Use :meth:`DataFrame.into_view` or
+        :meth:`TableProvider.from_dataframe`.
+        """
         from datafusion.dataframe import DataFrame as DataFrameWrapper
 
         if isinstance(df, DataFrameWrapper):