thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-17 21:24:28 +00:00
Code Issues Packages Projects Releases Wiki Activity

One more fix.

Browse Source
This commit is contained in:
Vladimir Chebotarev 2021-08-12 10:36:08 +03:00
parent 06476089d9
commit 40bedd88e7
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/Interpreters/MutationsInterpreter.cpp
View File

@ -200,13 +200,14 @@ bool isStorageTouchedByMutations(
else else
all_commands_can_be_skipped = false; all_commands_can_be_skipped = false;
if (command.type == MutationCommand::MATERIALIZE_COLUMN && !command.materialize_column_final) if (command.type == MutationCommand::MATERIALIZE_COLUMN)
{ {
if (!storage_from_merge_tree_data_part) if (!storage_from_merge_tree_data_part)
throw Exception("ALTER MATERIALIZE COLUMN is not supported for non-MergeTree tables", ErrorCodes::NOT_IMPLEMENTED); throw Exception("ALTER MATERIALIZE COLUMN is not supported for non-MergeTree tables", ErrorCodes::NOT_IMPLEMENTED);
if (storage_from_merge_tree_data_part->isColumnEmpty(command.column_name)) if (!storage_from_merge_tree_data_part->isColumnEmpty(command.column_name) || command.materialize_column_final)
all_commands_can_be_skipped = false; all_commands_can_be_skipped = false;
/// FIXME this is very bad place to check final :(((
} }
} }