lidavidm commented on code in PR #3556:
URL: https://github.com/apache/arrow-adbc/pull/3556#discussion_r2423377394


##########
python/adbc_driver_manager/adbc_driver_manager/dbapi.py:
##########


Review Comment:
   Hmm, I think this works but would be inconvenient for people (I'm guessing 
this will require type assertions all over the place). But I'm not sure of a 
better way to do this for now. (IIRC associated types are unsupported? Else we 
could template Cursor on the backend type and make the return type dependent on 
the backend type's associated type.)



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