Skip to content

Fix possible deadlock in OvercommitTracker during logging#37299

Merged
novikd merged 3 commits intomasterfrom
fix-overcommit-deadlock
May 18, 2022
Merged

Fix possible deadlock in OvercommitTracker during logging#37299
novikd merged 3 commits intomasterfrom
fix-overcommit-deadlock

Conversation

@novikd
Copy link
Copy Markdown
Member

@novikd novikd commented May 17, 2022

Changelog category (leave one):

  • Bug Fix (user-visible misbehavior in official stable or prestable release)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Fix possible deadlock in OvercommitTracker during logging. cc @alesapin @tavplubix
Fixes #37272

@davenger davenger self-assigned this May 17, 2022
@novikd
Copy link
Copy Markdown
Member Author

novikd commented May 18, 2022

Stateless tests (aarch64, actions) — fail: 1, passed: 4084, skipped: 16

Known bug #37133

Stress test (memory, actions) — Backward compatibility check: Server failed to start
Stress test (thread, actions) — Fatal message in clickhouse-server.log (see fatal_messages.txt)

Unrelated

@novikd novikd merged commit 5f37699 into master May 18, 2022
@novikd novikd deleted the fix-overcommit-deadlock branch May 18, 2022 10:01
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.

lock-order-inversion in OvercommitTracker

3 participants