ianmcook commented on code in PR #4525: URL: https://github.com/apache/arrow-adbc/pull/4525#discussion_r3605871318
########## docs/source/javascript/quickstart.rst: ########## @@ -36,13 +35,36 @@ Installation .. note:: - Node.js 22+, Deno 2.0+, or Bun 1.1+ are required. + Node.js 22+, Deno 2.0+, or Bun 1.1+ are required. This does not work in the + browser: the driver manager loads driver shared libraries from the + filesystem, which browsers cannot do. -To use any driver with the JavaScript driver manager, you'll need install the -appropriate driver shared library for your platform separately and pass the -absolute path to the driver manager. +To use any driver with the JavaScript driver manager, you need the driver's +shared library on your machine. -Some examples for the SQLite driver are provided below for convenience: +The easiest way to get it is with `dbc <https://docs.columnar.tech/dbc/>`__, +which installs prebuilt driver binaries from Columnar's `ADBC driver registry +<https://dbc-cdn.columnar.tech>`__: Review Comment: SGTM. I'll implement that for JavaScript and Python. -- 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]
