Skip to content

GH-50876: [C++][FS][Azure] Wait for Azurite to start - #50878

Open
booxter wants to merge 1 commit into
apache:mainfrom
booxter:main
Open

GH-50876: [C++][FS][Azure] Wait for Azurite to start#50878
booxter wants to merge 1 commit into
apache:mainfrom
booxter:main

Conversation

@booxter

@booxter booxter commented Aug 15, 2026

Copy link
Copy Markdown

Rationale for this change

Tests may fail due to race between process start and when it opens the port.

What changes are included in this PR?

Wait for the process to open the port before proceeding with the test.

AI disclosure: Codex was used to root cause the issue and draft the initial
version of the patch that was then iterated on by a human (me).

Are these changes tested?

Yes, I built and ran the test suite locally.

Are there any user-facing changes?

No.

@booxter
booxter requested a review from pitrou as a code owner August 15, 2026 16:29
@github-actions

Copy link
Copy Markdown

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

@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #50876 has no components, please add labels for components.

Comment thread cpp/src/arrow/filesystem/azurefs_test.cc
This should help with flakiness.

Fixes apache#50876

Assisted-by: Codex
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants