thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-09-24 02:30:51 +00:00
Code Issues Packages Projects Releases Wiki Activity

Slightly less garbage

Browse Source
This commit is contained in:
alesapin 2020-05-19 13:03:05 +03:00
parent 70e5553204
commit 111c576579
1 changed files with 1 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/Storages/MergeTree/MergeTreeDataMergerMutator.cpp
View File

@ -1266,16 +1266,6 @@ void MergeTreeDataMergerMutator::splitMutationCommands(
NameSet already_changed_columns;
bool is_compact_part = isCompactPart(part);
ColumnsDescription part_columns(part->getColumns());
NameSet modified_columns;
for (const auto & command : commands)
{
if (command.type == MutationCommand::Type::DELETE || command.type == MutationCommand::Type::UPDATE
|| command.type == MutationCommand::Type::MATERIALIZE_INDEX || command.type == MutationCommand::Type::MATERIALIZE_TTL || command.type == MutationCommand::Type::READ_COLUMN)
{
modified_columns.emplace(command.column_name);
}
}
for (const auto & command : commands)
{
@ -1311,7 +1301,7 @@ void MergeTreeDataMergerMutator::splitMutationCommands(
}
else if (command.type == MutationCommand::Type::RENAME_COLUMN && part_columns.has(command.column_name))
{
if (is_compact_part || modified_columns.count(command.rename_to))
if (is_compact_part)
{
for_interpreter.push_back(
{