amoeba commented on code in PR #4525: URL: https://github.com/apache/arrow-adbc/pull/4525#discussion_r3600331410
########## docs/source/cpp/recipe/quickstart.cc: ########## @@ -32,9 +32,33 @@ /// /// .. code-block:: shell /// -/// mamba install cmake compilers libadbc-driver-manager libadbc-driver-sqlite +/// mamba install cmake compilers libadbc-driver-manager +/// +/// Installing Drivers +/// ================== +/// +/// You'll need to install an ADBC driver for the database you want to +/// connect to. The easiest way is using dbc_: +/// +/// .. code-block:: shell +/// +/// # Install dbc +/// curl -fsSL https://dbc.sh | sh Review Comment: Yep. This should be gone now. Anywhere we need to tell users to install a driver with dbc should use this language: ```rst Installing Drivers ------------------ See :ref:`driver-table-install` for instructions on installing ADBC drivers for the database you want to connect to. For the example below, you could install `dbc <https://docs.columnar.tech/dbc>`__ and run ``dbc install postgresql``. ``` and we can easily tweak it everywhere if we want something different. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
