lidavidm commented on PR #3537:
URL: https://github.com/apache/arrow-adbc/pull/3537#issuecomment-3404169703

   > This looks good but I ran into one issue. If you do something like,
   > 
   > ```python
   > dbapi.connect("sqlite://games.sqlite")
   > ```
   > 
   > The correct driver is detected but, since the full URI string is passed 
unmodified as the "uri", and I assume the ADBC SQLite driver passes _that_ 
through, you get,
   > 
   > ```
   > adbc_driver_manager.OperationalError: IO: failed to open 
'sqlite://games.sqlite': failed to allocate memory
   > ```
   > 
   > Ultimately, sqlite itself doesn't handle such URIs so I don't think any 
change is needed but I thought I'd point it out in case you had any good ideas. 
Other drivers, such as postgresql, handle the unmodified URIs just fine.
   
   Well, this was discussed. Not everything uses URIs, and not every URI is 
self-identifying (sqlite uses `file:///`), so that's a limitation unless we're 
willing to have drivers add their own non-standard URI formats.


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