thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-25 00:52:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #65184 from ClickHouse/tavplubix-patch-14

Browse Source 
Add an assertion in ReplicatedMergeTreeQueue
This commit is contained in:
Alexander Tokmakov 2024-06-13 13:02:50 +00:00 committed by GitHub
commit c24e905bc6
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/Storages/MergeTree/ReplicatedMergeTreeQueue.cpp
View File

@ -2004,8 +2004,7 @@ MutationCommands ReplicatedMergeTreeQueue::getMutationCommands(
MutationCommands commands; MutationCommands commands;
for (auto it = begin; it != end; ++it) for (auto it = begin; it != end; ++it)
{ {
/// FIXME uncomment this assertion after relesing 23.5 (currently it fails in Upgrade check) chassert(mutation_pointer < it->second->entry->znode_name);
/// chassert(mutation_pointer < it->second->entry->znode_name);
mutation_ids.push_back(it->second->entry->znode_name); mutation_ids.push_back(it->second->entry->znode_name);
const auto & commands_from_entry = it->second->entry->commands; const auto & commands_from_entry = it->second->entry->commands;
commands.insert(commands.end(), commands_from_entry.begin(), commands_from_entry.end()); commands.insert(commands.end(), commands_from_entry.begin(), commands_from_entry.end());