Skip to content

Join the spawned handles in tests - #1165

Merged
Jay Bosamiya (Microsoft) (jaybosamiya-ms) merged 1 commit into
mainfrom
jayb/deadlock-fix-windows
Aug 13, 2026
Merged

Join the spawned handles in tests#1165
Jay Bosamiya (Microsoft) (jaybosamiya-ms) merged 1 commit into
mainfrom
jayb/deadlock-fix-windows

Conversation

@jaybosamiya-ms

Copy link
Copy Markdown
Member

This PR fixes some rare deadlocks that occur on Windows. Specifically, a still-running thread races with process teardown, which on Windows can in rare cases get stuck. Doing a join fixes this. It also helps surface panics if they occur on the thread.

@jaybosamiya-ms
Jay Bosamiya (Microsoft) (jaybosamiya-ms) marked this pull request as ready for review August 12, 2026 23:02
@jaybosamiya-ms

Copy link
Copy Markdown
Member Author

Relevant example of failure that this PR should fix: https://github.com/microsoft/litebox/actions/runs/31642623868/job/94268620821

image

@github-actions

Copy link
Copy Markdown

🤖 SemverChecks 🤖 No breaking API changes detected

Note: this does not mean API is unchanged, or even that there are no breaking changes; simply, none of the detections triggered.

@wdcui Weidong Cui (wdcui) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Thanks.

Merged via the queue into main with commit f5750e2 Aug 13, 2026
14 checks passed
@jaybosamiya-ms
Jay Bosamiya (Microsoft) (jaybosamiya-ms) deleted the jayb/deadlock-fix-windows branch August 13, 2026 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants