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

Cleanup.

Browse Source
This commit is contained in:
Nikolai Kochetov 2024-11-12 09:40:51 +00:00
parent 92114f3c74
commit 0e2cd397d2
2 changed files with 2 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -50,17 +50,6 @@ void fillRequiredColumns(const ActionsDAG::Node * node, std::unordered_map<const
}
}
// void addToOutputsIfNotAlreadyAdded(ActionsDAG & dag, const ActionsDAG::Node * node)
// {
// auto & outputs = dag.getOutputs();
// auto it = outputs.begin();
// while (it != outputs.end() && *it != node)
// ++it;
// if (it == outputs.end())
// outputs.push_back(node);
// }
/// Stores information about a node that has already been cloned or added to one of the new DAGs.
/// This allows to avoid cloning the same sub-DAG into multiple step DAGs but reference previously cloned nodes from earlier steps.
struct DAGNodeRef
@ -92,7 +81,6 @@ const ActionsDAG::Node & addClonedDAGToDAG(
/// If the node is known from the previous steps, add it as an input, except for constants
if (original_dag_node->type != ActionsDAG::ActionType::COLUMN)
{
// addToOutputsIfNotAlreadyAdded(*node_ref.dag, node_ref.node);
node_ref.dag->getOutputs().push_back(node_ref.node);
const auto & new_node = new_dag->addInput(node_ref.node->result_name, node_ref.node->result_type);

2
tests/queries/0_stateless/03199_merge_filters_bug.sql
View File

@ -1,3 +1,5 @@
set allow_reorder_prewhere_conditions=0;
drop table if exists t1;
drop table if exists t2;