Skip to content

Don't allow to drop or rename version column#20300

Merged
alesapin merged 1 commit intomasterfrom
restrict_to_drop_version_column
Feb 11, 2021
Merged

Don't allow to drop or rename version column#20300
alesapin merged 1 commit intomasterfrom
restrict_to_drop_version_column

Conversation

@alesapin
Copy link
Copy Markdown
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):
Restrict to DROP or RENAME version column of *CollapsingMergeTree and ReplacingMergeTree table engines.

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Feb 10, 2021
@alexey-milovidov
Copy link
Copy Markdown
Member

What about Sign column?

@alexey-milovidov alexey-milovidov self-assigned this Feb 11, 2021
@alesapin
Copy link
Copy Markdown
Member Author

What about Sign column?

I'll check and make separate PR.

@alesapin alesapin merged commit 5645325 into master Feb 11, 2021
@alesapin alesapin deleted the restrict_to_drop_version_column branch February 11, 2021 08:04
alesapin added a commit that referenced this pull request Feb 12, 2021
Backport #20300 to 21.2: Don't allow to drop or rename version column
alesapin added a commit that referenced this pull request Feb 12, 2021
Backport #20300 to 20.12: Don't allow to drop or rename version column
alesapin added a commit that referenced this pull request Feb 12, 2021
Backport #20300 to 21.1: Don't allow to drop or rename version column
@alesapin
Copy link
Copy Markdown
Member Author

#20398

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