flow better

Author: kevinjqliu

docs/source/user-guide/upgrade-guides.rst

@@ -52,38 +52,6 @@ parameter, which is a Python object that can be used to extract the
 ``FFI_LogicalExtensionCodec`` that is necessary.
 
 A complete example can be found in the `FFI example <https://github.com/apache/datafusion-python/tree/main/examples/datafusion-ffi-example>`_.
-Your methods need to be updated to take an additional parameter like in this
-example.
-
-For example, the ``__datafusion_catalog_provider__`` signature changed in
-DataFusion 52 from:
-
-.. code-block:: rust
-
-    pub fn __datafusion_catalog_provider__<'py>(
-        &self,
-        py: Python<'py>,
-    ) -> PyResult<Bound<'py, PyCapsule>>
-
-to:
-
-.. code-block:: rust
-
-    pub fn __datafusion_catalog_provider__<'py>(
-        &self,
-        py: Python<'py>,
-        session: Bound<PyAny>,
-    ) -> PyResult<Bound<'py, PyCapsule>>
-
-The same additional ``session: Bound<PyAny>`` parameter is required for these
-FFI hook methods in DataFusion 52:
-
-- ``__datafusion_table_provider__``
-- ``__datafusion_schema_provider__``
-- ``__datafusion_table_function__``
-
-If your hook still uses the old no-``session`` signature, update it to accept
-``session``.
 
 .. code-block:: rust
 
@@ -129,6 +97,38 @@ can implement a helper method such as:
     }
 
 
+Your methods need to be updated to take an additional ``session`` parameter.
+For example, the ``__datafusion_catalog_provider__`` signature changed in
+DataFusion 52 from:
+
+.. code-block:: rust
+
+    pub fn __datafusion_catalog_provider__<'py>(
+        &self,
+        py: Python<'py>,
+    ) -> PyResult<Bound<'py, PyCapsule>>
+
+to:
+
+.. code-block:: rust
+
+    pub fn __datafusion_catalog_provider__<'py>(
+        &self,
+        py: Python<'py>,
+        session: Bound<PyAny>,
+    ) -> PyResult<Bound<'py, PyCapsule>>
+
+The same additional ``session: Bound<PyAny>`` parameter is required for these
+FFI hook methods in DataFusion 52:
+
+- ``__datafusion_table_provider__``
+- ``__datafusion_schema_provider__``
+- ``__datafusion_table_function__``
+
+If your hook still uses the old no-``session`` signature, update it to accept
+``session``.
+
+
 The DataFusion FFI interface updates no longer depend directly on the
 ``datafusion`` core crate. You can improve your build times and potentially
 reduce your library binary size by removing this dependency and instead