lidavidm commented on code in PR #681:
URL: https://github.com/apache/arrow-adbc/pull/681#discussion_r1196707151
##########
adbc.h:
##########
@@ -669,6 +675,16 @@ struct ADBC_EXPORT AdbcDriver {
size_t, struct AdbcError*);
};
+/// \brief An instance of an initialized database driver (API 1.1.0).
+///
+/// This provides a common interface for vendor-specific driver
+/// initialization routines. Drivers should populate this struct, and
+/// applications can call ADBC functions through this struct, without
+/// worrying about multiple definitions of the same symbol.
+struct ADBC_EXPORT AdbcDriver110 {
+ struct AdbcDriver base;
+};
Review Comment:
If there's appetite for 2.0.0, we could do that. Most of the proposals so
far are not drastic changes: https://github.com/apache/arrow-adbc/milestone/3
except for possibly arbitrarily nested database schemas (which doesn't fit well
into the current nested data hierarchy from GetObjects, since Arrow doesn't
have recursive types).
We could use the driver manager to still provide compatibility if we want to
do 2.0.0.
--
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]
