CurtHagenlocher commented on PR #2388: URL: https://github.com/apache/arrow-adbc/pull/2388#issuecomment-2666791367
A little bit to Matt's point, it feels off to me to have the core ADBC classes derive from TracingBase in that 1) There's no integration with imported drivers, and 2) It exposes an implementation detail via the public API Is there already a sample implementation in e.g. the Spark driver so we can see how the usage pattern looks? -- 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