Skip to content

GH-49651: [C++][FlightRPC] Fix ODBC Linux test segmentation fault#49688

Open
alinaliBQ wants to merge 1 commit intoapache:mainfrom
Bit-Quill:gh-49651-linux-segfault
Open

GH-49651: [C++][FlightRPC] Fix ODBC Linux test segmentation fault#49688
alinaliBQ wants to merge 1 commit intoapache:mainfrom
Bit-Quill:gh-49651-linux-segfault

Conversation

@alinaliBQ
Copy link
Copy Markdown
Collaborator

@alinaliBQ alinaliBQ commented Apr 7, 2026

Rationale for this change

GH-49651

What changes are included in this PR?

Pass ARROW_TEST_LINKAGE: "static" to Linux ODBC tests.
Update CMakeLists.txt to enable static test linkage on Linux.

Note: enabling Linux test build will be in a separate PR

Are these changes tested?

Tested on local machine and in our local repository

Are there any user-facing changes?

N/A

* Resolve segfault issue on Linux and pass `ARROW_TEST_LINKAGE=static`

Pass `ARROW_TEST_LINKAGE=static`

* Clean up
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

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

@alinaliBQ alinaliBQ marked this pull request as ready for review April 7, 2026 22:44
@alinaliBQ
Copy link
Copy Markdown
Collaborator Author

@lidavidm @kou This PR is ready for review, please have a look

@lidavidm
Copy link
Copy Markdown
Member

lidavidm commented Apr 8, 2026

Hmm, why are there two copies of gRPC in the first place?

@alinaliBQ
Copy link
Copy Markdown
Collaborator Author

Hmm, why are there two copies of gRPC in the first place?

My understanding is, the arrow_flight_sql_odbc_shared has a copy of gRPC, and the tests have a copy of gRPC because it uses gRPC to set up the mock server. So when we dynamically link the tests to the shared ODBC lib, we end up with 2 copies.

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.

2 participants