Remove partition from statistics

Author: nuno-faria

datafusion-federation/src/sql/executor.rs

@@ -47,12 +47,7 @@ pub trait SQLExecutor: Sync + Send {
     /// Returns statistics for this `SQLExecutor` node. If statistics are not available, it should
     /// return [`Statistics::new_unknown`] (the default), not an error. See the `ExecutionPlan`
     /// trait.
-    fn partition_statistics(
-        &self,
-        _partition: Option<usize>,
-        _query: &str,
-        schema: SchemaRef,
-    ) -> Result<Statistics> {
+    fn statistics(&self, _query: &str, schema: SchemaRef) -> Result<Statistics> {
         Ok(Statistics::new_unknown(&schema))
     }
 

datafusion-federation/src/sql/mod.rs

@@ -345,9 +345,8 @@ impl ExecutionPlan for VirtualExecutionPlan {
         &self.props
     }
 
-    fn partition_statistics(&self, partition: Option<usize>) -> Result<Statistics> {
-        self.executor
-            .partition_statistics(partition, &self.final_sql()?, self.schema())
+    fn partition_statistics(&self, _partition: Option<usize>) -> Result<Statistics> {
+        self.executor.statistics(&self.final_sql()?, self.schema())
     }
 }