fix: update from_dataframe method to return the correct type

Author: kosiew

python/tests/test_context.py

@@ -361,7 +361,7 @@ def test_register_table_with_dataframe_errors(ctx):
 
     assert (
         str(exc_info.value)
-        == "Expected a Table or TableProvider. Convert DataFrames with \"DataFrame.into_view()\" or \"TableProvider.from_dataframe()\"."
+        == 'Expected a Table or TableProvider. Convert DataFrames with "DataFrame.into_view()" or "TableProvider.from_dataframe()".'
     )
 
 

src/table.rs

@@ -79,9 +79,9 @@ impl PyTableProvider {
     ///
     /// This method simply delegates to `DataFrame.into_view`.
     #[staticmethod]
-    pub fn from_dataframe(df: &PyDataFrame) -> PyDataFusionResult<Self> {
+    pub fn from_dataframe(df: &PyDataFrame) -> Self {
         let table_provider = df.to_view_provider();
-        Ok(Self::new(table_provider))
+        Self::new(table_provider)
     }
 
     /// Create a `TableProvider` from a `DataFrame` by converting it into a view.
@@ -97,7 +97,7 @@ impl PyTableProvider {
                 py.get_type::<PyDeprecationWarning>(),
             ),
         )?;
-        Self::from_dataframe(df)
+        Ok(Self::from_dataframe(df))
     }
 
     fn __datafusion_table_provider__<'py>(