Skip to content
Snippets Groups Projects
Commit 71a4e73e authored by Denis Smirnov's avatar Denis Smirnov Committed by Denis Smirnov
Browse files

perf: remove expression transformations for projections

At the moment our grammar doesn't support any sub-queries in the
projection list. So it is a waste of CPU resourcies to transform
projection lists (a big performance penalty for projections with
a huge amount of columns).
parent 80e4bbc8
No related branches found
Tags 1.7.3
1 merge request!108perf: remove expression transformations for projections
......@@ -66,8 +66,8 @@ impl Plan {
}
/// Transforms all expression subtrees in the plan
/// for selection, projection and inner join nodes
/// using the defined function.
/// for selection and inner join nodes using the
/// defined function.
///
/// # Errors
/// - If failed to get the plan top node.
......@@ -83,10 +83,7 @@ impl Plan {
for id in &nodes {
let rel = self.get_relation_node(*id)?;
let new_tree_id = match rel {
Relational::Projection {
output: tree_id, ..
}
| Relational::Selection {
Relational::Selection {
filter: tree_id, ..
}
| Relational::InnerJoin {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment