Skip to content

Fix possible deadlock in cache#35378

Merged
kssenii merged 1 commit intoClickHouse:masterfrom
kssenii:fix-deadlock
Mar 18, 2022
Merged

Fix possible deadlock in cache#35378
kssenii merged 1 commit intoClickHouse:masterfrom
kssenii:fix-deadlock

Conversation

@kssenii
Copy link
Copy Markdown
Member

@kssenii kssenii commented Mar 17, 2022

Changelog category (leave one):

  • Bug Fix (user-visible misbehaviour 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 cache.

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Mar 17, 2022
@alesapin
Copy link
Copy Markdown
Member

alesapin commented Mar 17, 2022

Single-mutex deadlock.

@kssenii
Copy link
Copy Markdown
Member Author

kssenii commented Mar 18, 2022

Style Check (actions) — Check style failed.

Looks unrelared:

/ClickHouse/src/Functions/ztest.cpp:72:            throw Exception{
/ClickHouse/src/Functions/ztest.cpp:82:            throw Exception{
/ClickHouse/src/Functions/ztest.cpp:122:            throw Exception{

@kssenii kssenii merged commit f91ab86 into ClickHouse:master Mar 18, 2022
kssenii added a commit that referenced this pull request Mar 19, 2022
Backport #35378 to 22.3: Fix possible deadlock in cache
@Felixoid Felixoid added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-bugfix Pull request with bugfix, not backported by default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants