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

Restrict merges from wide to compact parts #18381

Merged
merged 2 commits into from
Dec 24, 2020

Conversation

CurtizJ
Copy link
Member

@CurtizJ CurtizJ commented Dec 23, 2020

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):
Restrict merges from wide to compact parts. In case of vertical merge it led to broken result part.

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Dec 23, 2020
@nikitamikhaylov nikitamikhaylov self-assigned this Dec 23, 2020
@filimonov
Copy link
Contributor

from wide to compact

Can't vertical merge happen on compact parts?

@CurtizJ
Copy link
Member Author

CurtizJ commented Dec 23, 2020

from wide to compact

Can't vertical merge happen on compact parts?

No.

@nikitamikhaylov
Copy link
Member

Arcadia check is fixed in master and I hope this PR won't break it down again.

@nikitamikhaylov nikitamikhaylov merged commit 6818b24 into ClickHouse:master Dec 24, 2020
CurtizJ added a commit that referenced this pull request Dec 25, 2020
Backport #18381 to 20.12: Restrict merges from wide to compact parts
CurtizJ added a commit that referenced this pull request Dec 25, 2020
Backport #18381 to 20.11: Restrict merges from wide to compact parts
CurtizJ added a commit that referenced this pull request Dec 25, 2020
Backport #18381 to 20.10: Restrict merges from wide to compact parts
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.

4 participants