Skip to content

Conversation

@alinaliBQ
Copy link
Contributor

@alinaliBQ alinaliBQ commented Nov 3, 2025

Rationale for this change

Support statement deallocation

What changes are included in this PR?

  • Implement SQLFreeStmt
  • Add tests

Are these changes tested?

Tested locally on MSVC CI

Are there any user-facing changes?

N/A

@github-actions
Copy link

github-actions bot commented Nov 3, 2025

⚠️ GitHub issue #47706 has been automatically assigned in GitHub to PR creator.

@github-actions github-actions bot added awaiting committer review Awaiting committer review and removed awaiting review Awaiting review labels Nov 6, 2025
@alinaliBQ alinaliBQ changed the title GH-47706: [C++][FlightRPC] WIP: ODBC SQLFreeStmt implementation GH-47706: [C++][FlightRPC] ODBC SQLFreeStmt implementation Nov 10, 2025
@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting committer review Awaiting committer review labels Nov 11, 2025
@alinaliBQ alinaliBQ force-pushed the gh-47706-sql-alloc-stmt branch from 9199958 to 6927482 Compare November 28, 2025 23:11
Add statement allocate tests

Co-Authored-By: rscales <[email protected]>
@alinaliBQ alinaliBQ force-pushed the gh-47706-sql-alloc-stmt branch from 6927482 to d1d88b3 Compare November 28, 2025 23:12
@alinaliBQ alinaliBQ marked this pull request as ready for review November 28, 2025 23:13
@lidavidm lidavidm merged commit 7363d52 into apache:main Nov 30, 2025
45 of 46 checks passed
@lidavidm lidavidm removed the awaiting merge Awaiting merge label Nov 30, 2025
@github-actions github-actions bot added the awaiting merge Awaiting merge label Nov 30, 2025
@conbench-apache-arrow
Copy link

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit 7363d52.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants