You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@datafusion.apache.org by al...@apache.org on 2025/05/24 10:56:31 UTC

(datafusion) branch main updated: Minor: Fix name() for FilterPushdown physical optimizer rule (#16175)

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 d4218fd715 Minor: Fix name() for FilterPushdown physical optimizer rule (#16175)
d4218fd715 is described below

commit d4218fd715aa65c9cc01a60fb3436dd017298801
Author: Adrian Garcia Badaracco <17...@users.noreply.github.com>
AuthorDate: Sat May 24 05:56:28 2025 -0500

    Minor: Fix name() for FilterPushdown physical optimizer rule (#16175)
    
    * Fix name() for FilterPushdown physical optimizer rule
    
    Typo that wasn't caught during review...
    
    * fix
---
 datafusion/physical-optimizer/src/filter_pushdown.rs | 2 +-
 datafusion/sqllogictest/test_files/explain.slt       | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/datafusion/physical-optimizer/src/filter_pushdown.rs b/datafusion/physical-optimizer/src/filter_pushdown.rs
index 8d23bfbda6..5b2d47106b 100644
--- a/datafusion/physical-optimizer/src/filter_pushdown.rs
+++ b/datafusion/physical-optimizer/src/filter_pushdown.rs
@@ -388,7 +388,7 @@ impl PhysicalOptimizerRule for FilterPushdown {
     }
 
     fn name(&self) -> &str {
-        "PushdownFilter"
+        "FilterPushdown"
     }
 
     fn schema_check(&self) -> bool {
diff --git a/datafusion/sqllogictest/test_files/explain.slt b/datafusion/sqllogictest/test_files/explain.slt
index 87ab9c7aef..6227a76455 100644
--- a/datafusion/sqllogictest/test_files/explain.slt
+++ b/datafusion/sqllogictest/test_files/explain.slt
@@ -231,7 +231,7 @@ physical_plan after OutputRequirements
 physical_plan after aggregate_statistics SAME TEXT AS ABOVE
 physical_plan after join_selection SAME TEXT AS ABOVE
 physical_plan after LimitedDistinctAggregation SAME TEXT AS ABOVE
-physical_plan after PushdownFilter SAME TEXT AS ABOVE
+physical_plan after FilterPushdown SAME TEXT AS ABOVE
 physical_plan after EnforceDistribution SAME TEXT AS ABOVE
 physical_plan after CombinePartialFinalAggregate SAME TEXT AS ABOVE
 physical_plan after EnforceSorting SAME TEXT AS ABOVE
@@ -306,7 +306,7 @@ physical_plan after OutputRequirements
 physical_plan after aggregate_statistics SAME TEXT AS ABOVE
 physical_plan after join_selection SAME TEXT AS ABOVE
 physical_plan after LimitedDistinctAggregation SAME TEXT AS ABOVE
-physical_plan after PushdownFilter SAME TEXT AS ABOVE
+physical_plan after FilterPushdown SAME TEXT AS ABOVE
 physical_plan after EnforceDistribution SAME TEXT AS ABOVE
 physical_plan after CombinePartialFinalAggregate SAME TEXT AS ABOVE
 physical_plan after EnforceSorting SAME TEXT AS ABOVE
@@ -347,7 +347,7 @@ physical_plan after OutputRequirements
 physical_plan after aggregate_statistics SAME TEXT AS ABOVE
 physical_plan after join_selection SAME TEXT AS ABOVE
 physical_plan after LimitedDistinctAggregation SAME TEXT AS ABOVE
-physical_plan after PushdownFilter SAME TEXT AS ABOVE
+physical_plan after FilterPushdown SAME TEXT AS ABOVE
 physical_plan after EnforceDistribution SAME TEXT AS ABOVE
 physical_plan after CombinePartialFinalAggregate SAME TEXT AS ABOVE
 physical_plan after EnforceSorting SAME TEXT AS ABOVE


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