Skip to content

Conversation

@alinaliBQ
Copy link
Contributor

@alinaliBQ alinaliBQ commented Nov 4, 2025

Rationale for this change

Add support for returning columns metadata from the connected data source.

What changes are included in this PR?

  • SQLColumns & Test

Are these changes tested?

Tested in local MSVC

Are there any user-facing changes?

N/A

@github-actions
Copy link

github-actions bot commented Nov 4, 2025

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

@github-actions github-actions bot added awaiting committer review Awaiting committer review awaiting changes Awaiting changes and removed awaiting review Awaiting review awaiting committer review Awaiting committer review labels Nov 4, 2025
@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting changes Awaiting changes labels Nov 6, 2025
@alinaliBQ alinaliBQ changed the title GH-47720: [C++][FlightRPC] WIP: ODBC Columns Metadata GH-47720: [C++][FlightRPC] ODBC Columns Metadata Nov 10, 2025
@alinaliBQ alinaliBQ force-pushed the gh-47720-sql-columns branch 2 times, most recently from 082eb6e to 9507ee2 Compare December 6, 2025 00:09
@alinaliBQ alinaliBQ marked this pull request as ready for review December 6, 2025 00:09
@alinaliBQ alinaliBQ force-pushed the gh-47720-sql-columns branch 2 times, most recently from 378a314 to 18f8939 Compare December 9, 2025 20:34
Comment on lines +185 to +187
if (skipping_test_) {
return;
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this used for?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Used for checking if the environment variable ARROW_FLIGHT_SQL_ODBC_CONN exists. If it doesn't exist, skipping_test_ is set to true and the test is skipped.

All issues mentioned in the TODO comments are completed. We forgot to remove the comments
@alinaliBQ alinaliBQ force-pushed the gh-47720-sql-columns branch from 94c9e99 to e0cd3b8 Compare December 13, 2025 00:14
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