Skip to content

Addition to #23997#24208

Merged
tavplubix merged 2 commits intomasterfrom
fix_intersecting_parts_again
May 18, 2021
Merged

Addition to #23997#24208
tavplubix merged 2 commits intomasterfrom
fix_intersecting_parts_again

Conversation

@tavplubix
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):

  • Not for changelog (changelog entry is not required)

Detailed description / Documentation draft:
Addition to #23997. Fixes another case of intersecting log entries.

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label May 17, 2021
@tavplubix tavplubix force-pushed the fix_intersecting_parts_again branch from 7abcf3e to 20b9af7 Compare May 17, 2021 21:24
@alesapin alesapin self-assigned this May 18, 2021
Copy link
Copy Markdown
Member

@alesapin alesapin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Discussed in tg -- rename drop_range_version to alter_partition_version. Maybe return removePartProducingOpsInRange after entry creation (optional).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants