Skip to content

GH-49463: [C++][FlightRPC] Add Ubuntu ODBC Support#49564

Open
alinaliBQ wants to merge 1 commit intoapache:mainfrom
Bit-Quill:gh-49463-odbc-linux
Open

GH-49463: [C++][FlightRPC] Add Ubuntu ODBC Support#49564
alinaliBQ wants to merge 1 commit intoapache:mainfrom
Bit-Quill:gh-49463-odbc-linux

Conversation

@alinaliBQ
Copy link
Collaborator

@alinaliBQ alinaliBQ commented Mar 19, 2026

Rationale for this change

GH-49463
Add Ubuntu support so users can connect using odbc on Linux.

What changes are included in this PR?

  • Enable Linux ODBC build with unicode support
  • Add ODBC Ubuntu build in CI
  • Added docker-compose for Flight SQL ODBC
  • Register ODBC after build
  • Replaced boost::lexicographical_compare with std::lexicographical_compare
  • Fixed conversion bugs in SetAttributeSQLWCHAR
  • Convert from std::string to std::u16string directly without involving wide string (wstring)
  • Enabling ODBC Linux test build will be added in a separate PR

Are these changes tested?

  • Ubuntu build is tested in CI
  • ODBC is tested with isql on local docker, and confirmed to be able to connect to online remote instance

Are there any user-facing changes?

N/A

- Enable Linux ODBC build with unicode support
- Added `docker-compose` for Flight SQL ODBC
- Register ODBC after build
- Replaced `boost::lexicographical_compare` with `std::lexicographical_compare`
- Fixed conversion bugs in `SetAttributeSQLWCHAR`
- Convert from std::string to std::u16string directly without involving wide string (wstring)
* Add patch for Windows build
@alinaliBQ alinaliBQ force-pushed the gh-49463-odbc-linux branch from 54b79f8 to 848fdfe Compare March 19, 2026 23:12
@alinaliBQ
Copy link
Collaborator Author

For ODBC release / debug AMD64 macOS 15-intel Workflow failure, I raised issue #49563

Error: Unexpected shared dependency found in /Users/runner/work/arrow/arrow/build/cpp/release/libarrow_flight_sql_odbc.dylib: `libunwind`

EXPECT_FALSE(std::wstring(message).empty());
}

// TODO: verify that `SQLGetConnectOption` is not required by Excel.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I cleaned up comment that should have been removed before.

@alinaliBQ alinaliBQ marked this pull request as ready for review March 20, 2026 17:37
@github-actions github-actions bot added awaiting committer review Awaiting committer review and removed awaiting review Awaiting review labels Mar 20, 2026
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.

1 participant