Address PR comments

Author: davisp

examples/datafusion-ffi-example/python/tests/_test_table_provider_factory.py

@@ -17,8 +17,6 @@
 
 from __future__ import annotations
 
-import pyarrow as pa
-import pytest
 from datafusion import SessionContext
 from datafusion_ffi_example import MyTableProviderFactory
 
@@ -35,7 +33,7 @@ def test_table_provider_factory_ffi() -> None:
         foo
         STORED AS my_format
         LOCATION '';
-    """)
+    """).collect()
 
     # Query the pre-populated table
     result = ctx.sql("SELECT * FROM foo;").collect()

python/datafusion/context.py

@@ -836,12 +836,12 @@ def register_table_factory(
     ) -> None:
         """Register a :py:class:`~datafusion.TableProviderFactoryExportable`.
 
-        The registered factory can be reference from SQL DDL statements executed
+        The registered factory can be referenced from SQL DDL statements executed
         against this context.
 
         Args:
             format: The value to be used in `STORED AS ${format}` clause.
-            factory: A PyCapsule that implements TableProviderFactoryExportable"
+            factory: A PyCapsule that implements :class:`TableProviderFactoryExportable`
         """
         self.ctx.register_table_factory(format, factory)