henryharbeck commented on issue #2413: URL: https://github.com/apache/arrow-adbc/issues/2413#issuecomment-2575278637
Thanks both for the prompt responses. For a little more context behind the request (if completed), I am wanting to propose to Polars that they also remove PyArrow as a required dependency for using the ADBC engine in database I/O. --- >I suppose it's doable enough to change it so that as long as you don't try and read result sets (which needs something to process the Arrow data) Upon reading the `adbc_driver_manager.dbapi.Cursor` Python docs again, I realise there is no current API to fetch the results as a PyCapsule. Would you consider this as an additional feature request? If so, happy to raise an additional issue for this. Otherwise I'll propose Polars use the lower-level API for database reads (which I don't imagine will be a deal-breaker by any stretch). >The proposal here would technically not implement DBAPI either, but I suppose it'd be closer! Sorry for any confusion, I was conflating DBAPI 2.0 (PEP 249) and the `adbc_driver_manager.dbapi` and `adbc_driver_<database>.dbapi` modules/namespaces under the term "DBAPI" >I figure if we can get away with no dependencies at all that might be useful too That would be ideal from my (and I dare say other libraries') POV, where some features/APIs require dependencies (e.g., `fetch_arrow_table`, `fetch_df`), but none are explicitly required by the library itself. -- 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]
