Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[7.2.0] Prevent currentThread().join() on cli thread crash #22368

Merged
merged 1 commit into from
May 14, 2024

Conversation

bazel-io
Copy link
Member

If the cli-update-thread is crashing, it may attempt to interrupt and join on itself. Hopefully no updateThread could be in stopUpdateThread without going through handleCrash() -> Event.FATAL sequence through BlazeRuntime.

Fixes #22051

Closes #22122.

PiperOrigin-RevId: 633653817
Change-Id: Iaef5df56358d74bd7210ad8cb3562b452de9eb6a

Commit 6306240

If the cli-update-thread is crashing, it may attempt to interrupt and join on itself. Hopefully no updateThread could be in stopUpdateThread without going through handleCrash() -> Event.FATAL sequence through BlazeRuntime.

Fixes bazelbuild#22051

Closes bazelbuild#22122.

PiperOrigin-RevId: 633653817
Change-Id: Iaef5df56358d74bd7210ad8cb3562b452de9eb6a
@bazel-io bazel-io requested a review from a team as a code owner May 14, 2024 18:38
@bazel-io bazel-io added awaiting-review PR is awaiting review from an assigned reviewer team-CLI Console UI labels May 14, 2024
@bazel-io bazel-io requested a review from tjgq May 14, 2024 18:38
@iancha1992 iancha1992 enabled auto-merge May 14, 2024 18:45
@iancha1992 iancha1992 added this pull request to the merge queue May 14, 2024
Merged via the queue into bazelbuild:release-7.2.0 with commit 594da0c May 14, 2024
33 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-CLI Console UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants