danepitkin opened a new pull request, #36559: URL: https://github.com/apache/arrow/pull/36559
### Rationale for this change Arrow introduced a `Result` return object that can be used in place of passing output parameters. Most of Flight has already been migrated. ### Are these changes tested? C++ tests pass, which involve the sqlite and acero example servers. Any other testing we would like to see? ### Are there any user-facing changes? Yes. The protected methods DoPut, DoGet, and DoAction for the SQL client have been updated. DoGet already exposed a public API and that has been updated to be virtual. -- 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
