refactor(cubesql): Remove Extension from LogicalPlanLanguage::CubeScan (

#8775) Extensions is used meaningfully only for CubeScan, but not for CubeScanWrapper. Both are first-class logical plan nodes for us, so Extension is not helpful in egraph
cube-js · Dec 6, 2024 · 060d666 · 060d666
1 parent 5b10a68
commit 060d666
Show file tree

Hide file tree

Showing 3 changed files with 468 additions and 530 deletions.
diff --git a/rust/cubesql/cubesql/src/compile/rewrite/analysis.rs b/rust/cubesql/cubesql/src/compile/rewrite/analysis.rs
@@ -1242,7 +1242,6 @@ impl LogicalPlanAnalysis {
                 Some(c.to_string())
             }
             LogicalPlanLanguage::CubeScan(params) => cube_reference(params[0]),
-            LogicalPlanLanguage::Extension(params) => cube_reference(params[0]),
             _ => None,
         }
     }