[GitHub] [arrow-adbc] lidavidm opened a new pull request, #27: Make AdbcConnectionNew 2-adic for consistency

[GitBox <gi...@apache.org>]

lidavidm opened a new pull request, #27:
URL: https://github.com/apache/arrow-adbc/pull/27

   Make AdbcConnectionNew consistent with AdbcDatabaseNew.
   
   This mostly doesn't affect the driver implementations; it does make the driver manager a little more complex.


-- 
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: issues-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow-adbc] lidavidm merged pull request #27: Make AdbcConnectionNew 2-adic for consistency

[GitBox <gi...@apache.org>]

lidavidm merged PR #27:
URL: https://github.com/apache/arrow-adbc/pull/27


-- 
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: issues-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow-adbc] lidavidm commented on pull request #27: Make AdbcConnectionNew 2-adic for consistency

[GitBox <gi...@apache.org>]

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

   CC @hannes. I've left `AdbcStatementNew` alone for now, since that one seems fully a wash (and it would mean we wouldn't be able to validate the query or anything until execution time)


-- 
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: issues-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org