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

Fixed metrics in BackgroundProcessingPool #8194

Merged
merged 2 commits into from
Dec 12, 2019
Merged

Fixed metrics in BackgroundProcessingPool #8194

merged 2 commits into from
Dec 12, 2019

Conversation

excitoon
Copy link
Contributor

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 (up to few sentences, required except for Non-significant/Documentation categories):

Fixed metrics in BackgroundProcessingPool (they were interfering with metrics of processing pool for moves and could throw exceptions invalid arguments passed to getMaxSourcePartsSize: pool_used > pool_size during merges and mutations if there is a storage policy set for a table).

@excitoon
Copy link
Contributor Author

This fixes problem introduced in #7670 .

@alexey-milovidov alexey-milovidov merged commit 32f42c4 into ClickHouse:master Dec 12, 2019
@KochetovNicolai KochetovNicolai added the pr-bugfix Pull request with bugfix, not backported by default label Dec 16, 2019
@KochetovNicolai
Copy link
Member

KochetovNicolai commented Dec 17, 2019

Conflicts with 19.16 and 19.17 and 19.18

@excitoon
Copy link
Contributor Author

@KochetovNicolai there is no need to backport it to 19.16 because 19.16 has no pool for background moves.

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.

None yet

4 participants