refactor(cubesql): Tighten rule for non-trivial wrapper pullup

cube-js · Nov 12, 2024 · 59d1231 · 59d1231
1 parent 95526cb
commit 59d1231
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/rust/cubesql/cubesql/src/compile/rewrite/rules/wrapper/wrapper_pull_up.rs b/rust/cubesql/cubesql/src/compile/rewrite/rules/wrapper/wrapper_pull_up.rs
@@ -204,7 +204,8 @@ impl WrapperRules {
                         ),
                         "?select_alias",
                         "?select_distinct",
-                        "?select_push_to_cube",
+                        // This node has a WrappedSelect in from, so it's not allowed to use push to Cube
+                        "WrappedSelectPushToCube:false",
                         "?select_ungrouped_scan",
                     ),
                     "CubeScanWrapperFinalized:false",
@@ -245,7 +246,7 @@ impl WrapperRules {
                             "?order_expr",
                             "?select_alias",
                             "?select_distinct",
-                            "?select_push_to_cube",
+                            "WrappedSelectPushToCube:false",
                             "?select_ungrouped_scan",
                         ),
                         "?alias_to_cube",