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

Fix race on DDLWorker::is_circular_replicated #17640

Merged
merged 1 commit into from
Dec 1, 2020

Conversation

tavplubix
Copy link
Member

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Changelog category (leave one):

  • Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
In might be determined incorrectly if cluster is circular- (cross-) replicated or not when executing ON CLUSTER query due to race condition when pool_size > 1. It's fixed.

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Nov 30, 2020
@alexey-milovidov alexey-milovidov self-assigned this Nov 30, 2020
@tavplubix
Copy link
Member Author

Integration tests - known issue
ClickHouse special build check - pull rate limit, known issue
Yandex synchronization check - broken in master

@tavplubix tavplubix merged commit a908b2d into master Dec 1, 2020
@tavplubix tavplubix deleted the fix_ddl_worker_is_circular branch December 1, 2020 14:43
tavplubix added a commit that referenced this pull request Dec 2, 2020
Backport #17640 to 20.11: Fix race on DDLWorker::is_circular_replicated
tavplubix added a commit that referenced this pull request Dec 2, 2020
Backport #17640 to 20.12: Fix race on DDLWorker::is_circular_replicated
tavplubix added a commit that referenced this pull request Dec 2, 2020
Backport #17640 to 20.10: Fix race on DDLWorker::is_circular_replicated
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants