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

Do not try convert columns that does not exists in the result block

Browse Source
This commit is contained in:
Azat Khuzhin 2021-06-02 10:31:28 +03:00
parent 01e6a4d919
commit 6cb2e81d48
1 changed files with 13 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/Storages/StorageMaterializedView.cpp
View File

@ -176,6 +176,19 @@ void StorageMaterializedView::read(
} }
target_header.erase(target_only_positions); target_header.erase(target_only_positions);
/// No need to convert columns that does not exists in the result header.
///
/// Distributed storage may process query up to the specific stage, and
/// so the result header may not include all the columns from the
/// materialized view.
std::set<size_t> source_only_positions;
for (const auto & column : mv_header)
{
if (!target_header.has(column.name))
source_only_positions.insert(mv_header.getPositionByName(column.name));
}
mv_header.erase(source_only_positions);
if (!blocksHaveEqualStructure(mv_header, target_header)) if (!blocksHaveEqualStructure(mv_header, target_header))
{ {
auto converting_actions = ActionsDAG::makeConvertingActions(target_header.getColumnsWithTypeAndName(), auto converting_actions = ActionsDAG::makeConvertingActions(target_header.getColumnsWithTypeAndName(),