(datafusion) branch main updated: Minor: make some physical_optimizer rules public (#11171)

[al...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

alamb pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion.git


The following commit(s) were added to refs/heads/main by this push:
     new 27d3aa6a62 Minor: make some physical_optimizer rules public (#11171)
27d3aa6a62 is described below

commit 27d3aa6a62aa94367013710cce23242ed0de423e
Author: Albert Skalt <13...@users.noreply.github.com>
AuthorDate: Sat Jun 29 20:57:52 2024 +0300

    Minor: make some physical_optimizer rules public (#11171)
    
    This patch makes public some physical_optimizer submodules. This allows
    external code to use optimizations passes from these submodules.
---
 datafusion/core/src/physical_optimizer/mod.rs               | 4 ++--
 datafusion/core/src/physical_optimizer/update_aggr_exprs.rs | 1 +
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/datafusion/core/src/physical_optimizer/mod.rs b/datafusion/core/src/physical_optimizer/mod.rs
index 7cc9a0fb75..e1bde36bd6 100644
--- a/datafusion/core/src/physical_optimizer/mod.rs
+++ b/datafusion/core/src/physical_optimizer/mod.rs
@@ -31,12 +31,12 @@ pub mod limited_distinct_aggregation;
 pub mod optimizer;
 pub mod output_requirements;
 pub mod pipeline_checker;
-mod projection_pushdown;
+pub mod projection_pushdown;
 pub mod pruning;
 pub mod replace_with_order_preserving_variants;
 mod sort_pushdown;
 pub mod topk_aggregation;
-mod update_aggr_exprs;
+pub mod update_aggr_exprs;
 mod utils;
 
 #[cfg(test)]
diff --git a/datafusion/core/src/physical_optimizer/update_aggr_exprs.rs b/datafusion/core/src/physical_optimizer/update_aggr_exprs.rs
index 6a6ca815c5..1ad4179cef 100644
--- a/datafusion/core/src/physical_optimizer/update_aggr_exprs.rs
+++ b/datafusion/core/src/physical_optimizer/update_aggr_exprs.rs
@@ -52,6 +52,7 @@ use datafusion_physical_plan::{
 pub struct OptimizeAggregateOrder {}
 
 impl OptimizeAggregateOrder {
+    #[allow(missing_docs)]
     pub fn new() -> Self {
         Self::default()
     }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@datafusion.apache.org
For additional commands, e-mail: commits-help@datafusion.apache.org