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


##########
docs/source/glossary.rst:
##########
@@ -46,16 +46,24 @@ Glossary
      protocol, but exposes the JDBC client API instead.

Review Comment:
   It'd be nice to expand this definition of **driver** to better explain what 
a driver is in ADBC specifically. Maybe add a second paragraph in this 
definition that says something like:
   
   ```rst
        ADBC drivers can be implemented as libraries in different languages
        including C++, C#, Go, and Rust. A driver can be imported directly into
        an application that's implemented in the same language, or it can be
        compiled into a shared library (a ``.so`` file for Linux, a ``.dylib`` 
file
        for macOS, or a ``.dll`` file for Windows) and dynamically loaded into
        an application in any supported language using a :term:`driver manager`.
   ```



-- 
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]

Reply via email to