GH-39774: [Go] Add public access to PreparedStatement handle#39775
GH-39774: [Go] Add public access to PreparedStatement handle#39775zeroshade merged 1 commit intoapache:mainfrom
Conversation
|
Thanks for opening a pull request! If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project. Then could you also rename the pull request title in the following format? or In the case of PARQUET issues on JIRA the title also supports: See also: |
|
|
|
Would it make sense to add a small unit test to just ensure that it is populated and returned properly? |
I updated the current client test to validate the value of the prepared statement handle property. |
|
Thanks @abandy! |
|
After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit c97e6c4. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
…pache#39775) * Closes: apache#39774 Authored-by: Alva Bandy <abandy@live.com> Signed-off-by: Matt Topol <zotthewizard@gmail.com>
Uh oh!
There was an error while loading. Please reload this page.