Skip to content

Fix LogicalError if parallel queries are trying to acquire single lock#86751

Merged
GrigoryPervakov merged 1 commit intomasterfrom
pervakovg/parallel_with_lock
Sep 11, 2025
Merged

Fix LogicalError if parallel queries are trying to acquire single lock#86751
GrigoryPervakov merged 1 commit intomasterfrom
pervakovg/parallel_with_lock

Conversation

@GrigoryPervakov
Copy link
Copy Markdown
Member

@GrigoryPervakov GrigoryPervakov commented Sep 5, 2025

Before the patch, parallel with queries shared the same query_id used to acquire the table drop lock.

Closes: #79413

Changelog category (leave one):

Bug Fix (user-visible misbehavior in an official stable release)

Changelog entry (a [user-readable short description]

Fix LogicalError if parallel queries are trying to acquire single lock

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Sep 5, 2025

Workflow [PR], commit [c9f03b0]

Summary:

@clickhouse-gh clickhouse-gh bot added the pr-bugfix Pull request with bugfix, not backported by default label Sep 5, 2025
@vitlibar vitlibar self-assigned this Sep 8, 2025
@GrigoryPervakov GrigoryPervakov marked this pull request as ready for review September 9, 2025 15:00
@GrigoryPervakov
Copy link
Copy Markdown
Member Author

Failed sync CI tests are known flaky

@GrigoryPervakov GrigoryPervakov added this pull request to the merge queue Sep 10, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Sep 10, 2025
@GrigoryPervakov GrigoryPervakov added this pull request to the merge queue Sep 11, 2025
Merged via the queue into master with commit 55540f5 Sep 11, 2025
122 checks passed
@GrigoryPervakov GrigoryPervakov deleted the pervakovg/parallel_with_lock branch September 11, 2025 13:42
@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-synced-to-cloud The PR is synced to the cloud repo label Sep 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Pull request with bugfix, not backported by default pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Logical error: Cannot acquire exclusive lock while RWLock is already locked

3 participants