WillAyd commented on code in PR #3362:
URL: https://github.com/apache/arrow-adbc/pull/3362#discussion_r2308860877
##########
python/adbc_driver_manager/adbc_driver_manager/dbapi.py:
##########
@@ -721,10 +733,17 @@ def execute(self, operation: Union[bytes, str],
parameters=None) -> None:
The query to execute. Pass SQL queries as strings,
(serialized) Substrait plans as bytes.
parameters
- Parameters to bind. Can be a Python sequence (to provide
- a single set of parameters), or an Arrow record batch,
- table, or record batch reader (to provide multiple
- parameters, which will each be bound in turn).
+ Parameters to bind. Can be a Python sequence (to bind a single
+ set of parameters), a Python dictionary (to bind a single set of
Review Comment:
Does providing a dictionary automatically set the option to bind by name? If
not, should it?
--
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
