amoeba commented on code in PR #3388:
URL: https://github.com/apache/arrow-adbc/pull/3388#discussion_r2320009216
##########
docs/source/glossary.rst:
##########
@@ -46,16 +46,24 @@ Glossary
protocol, but exposes the JDBC client API instead.
driver manager
- A library that helps manage multiple drivers for a given client API.
- For example, the JDBC driver manager can find a appropriate driver
- implementation for a database URI.
+ A library for loading and using :term:`drivers <driver>`. A :term:`driver
+ manager` implements the ADBC API and delegates to
+ dynamically-loaded drivers. It simplifies using multiple drivers in a
+ single application and makes it possible to use drivers written in any
+ language, regardless of the language the application is written in.
- The ADBC driver manager in each language is similar. In C/C++, it can
+ The :term:`driver manager` in each language is similar. In C/C++, it can
dynamically load drivers so that applications do not have to directly
link to them. (Since all drivers expose the same API, their symbols
would collide otherwise.) In Python, it loads drivers and provides
Python bindings on top.
+ driver manifest
+ A file (in TOML format) describing a :term:`driver`. This file's structure
+ is part of the ADBC :doc:`specification <format/specification>`. A
+ :term:`driver manager` can load a driver from a :term:`driver manifest`
Review Comment:
For some reason I like it but it's fine to remove. I found another instance
so I'll also removed that.
Done in 8af4eb8d1.
--
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
