mirror of
https://github.com/ClickHouse/ClickHouse.git
synced 2024-09-19 16:20:50 +00:00
Merge remote-tracking branch 'origin/pr-plan-rewrite' into pr-local-plan
This commit is contained in:
commit
4916e3376e
@ -890,10 +890,19 @@ JoinTreeQueryPlan buildQueryPlanForTableExpression(QueryTreeNodePtr table_expres
|
|||||||
if (reading)
|
if (reading)
|
||||||
break;
|
break;
|
||||||
|
|
||||||
|
QueryPlan::Node * prev_node = node;
|
||||||
if (!node->children.empty())
|
if (!node->children.empty())
|
||||||
|
{
|
||||||
|
chassert(node->children.size() == 1);
|
||||||
node = node->children.at(0);
|
node = node->children.at(0);
|
||||||
|
}
|
||||||
else
|
else
|
||||||
node = nullptr;
|
{
|
||||||
|
if (prev_node->step)
|
||||||
|
throw Exception(ErrorCodes::LOGICAL_ERROR, "Step is expected to be ReadFromMergeTree but it's {}", prev_node->step->getName());
|
||||||
|
else
|
||||||
|
throw Exception(ErrorCodes::LOGICAL_ERROR, "Step is expected to be ReadFromMergeTree, and wtf - last node with empty step");
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
chassert(reading);
|
chassert(reading);
|
||||||
@ -948,9 +957,6 @@ JoinTreeQueryPlan buildQueryPlanForTableExpression(QueryTreeNodePtr table_expres
|
|||||||
table_expression_query_info.storage_limits,
|
table_expression_query_info.storage_limits,
|
||||||
std::move(reading_step));
|
std::move(reading_step));
|
||||||
query_plan = std::move(query_plan_parallel_replicas);
|
query_plan = std::move(query_plan_parallel_replicas);
|
||||||
|
|
||||||
const Block & query_plan_header = query_plan.getCurrentDataStream().header;
|
|
||||||
LOG_DEBUG(getLogger(__PRETTY_FUNCTION__), "Parallel replicas query_plan_header:\n{}", query_plan_header.dumpStructure());
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user