ianmcook commented on code in PR #3388:
URL: https://github.com/apache/arrow-adbc/pull/3388#discussion_r2317196967


##########
docs/source/python/driver_manager.rst:
##########
@@ -43,34 +39,52 @@ Installation
 Usage
 =====
 
-.. warning:: This API is for low level usage only.  **You almost certainly
-             should not use this**, instead use the entrypoints provided by
-             driver packages, for example:
+Using the driver manager is different from using the individual, 
driver-specific
+packages such as ``adbc_driver_postgresql``.
 
-             - :func:`adbc_driver_sqlite.dbapi.connect`
-             - :func:`adbc_driver_sqlite.connect`
+With the driver-specific packages, you connect to the target database with the
+``connect`` method provided by the package you're using.  For example,
+:func:`adbc_driver_postgresql.connect` or :func:`adbc_driver_sqlite.connect`.
 
-The Python bindings for each driver abstract the steps here for you behind a
-convenient ``connect`` function.  For example, prefer
-:func:`adbc_driver_sqlite.connect` or :func:`adbc_driver_postgresql.connect`
-to manually constructing the connection as demonstrated here.
+With the driver manager package, you use a single package and API regardless of
+the database you're connecting to.

Review Comment:
   The other key difference here is that:
   - With the driver-specific packages, the driver shared library 
(`.so`/`.dylib`/`.dll` file) is included in the package.
   - With the driver manager, the driver shared library is installed separately.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to