ianmcook commented on code in PR #3388: URL: https://github.com/apache/arrow-adbc/pull/3388#discussion_r2317376317
########## 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: Looks great -- 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
