override supports_rewrite for FederationOptimizerRule and SQLFederati…

…onOptimizerRule
datafusion-contrib · Aug 23, 2024 · ac84a50 · ac84a50
1 parent d9449e5
commit ac84a50
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/datafusion-federation/src/optimizer.rs b/datafusion-federation/src/optimizer.rs
@@ -35,6 +35,12 @@ impl OptimizerRule for FederationOptimizerRule {
     fn name(&self) -> &str {
         "federation_optimizer_rule"
     }
+
+    /// XXX
+    /// Does this rule support rewriting owned plans (rather than by reference)?
+    fn supports_rewrite(&self) -> bool {
+        false
+    }
 }
 
 enum ScanResult {

diff --git a/datafusion-federation/src/sql/mod.rs b/datafusion-federation/src/sql/mod.rs
@@ -93,6 +93,12 @@ impl OptimizerRule for SQLFederationOptimizerRule {
     fn name(&self) -> &str {
         "federate_sql"
     }
+
+    /// XXX
+    /// Does this rule support rewriting owned plans (rather than by reference)?
+    fn supports_rewrite(&self) -> bool {
+        false
+    }
 }
 struct SQLFederationPlanner {
     executor: Arc<dyn SQLExecutor>,