CurtHagenlocher commented on PR #1821:
URL: https://github.com/apache/arrow-adbc/pull/1821#issuecomment-2103424427
If we want to pass all the initial options in at once when we construct
something, then they kind of have to be represented as "object". Post
construction, it might be justified to have separate overloads for setting
different types but I also feel like this is an area where it's reasonable to
want to support both e.g. `SetOption("adbc.x", "true")` and
`SetOption("adbc.x", true)` for a bool-valued option that's officially
represented as a string. And if we're going to convert anyway... .
--
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]
