(calcite) branch main updated: [CALCITE-6897] AbstractConverter of root node is not needed in topdown mode

[cancai]

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

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


The following commit(s) were added to refs/heads/main by this push:
     new 6de592c79e [CALCITE-6897] AbstractConverter of root node is not needed 
in topdown mode
6de592c79e is described below

commit 6de592c79e7b6c60282632b948b3de7ffd7786f0
Author: Zhen Chen <czjour...@163.com>
AuthorDate: Tue Mar 18 10:15:31 2025 +0800

    [CALCITE-6897] AbstractConverter of root node is not needed in topdown mode
---
 core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java 
b/core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java
index aaf939f2e6..8f4cf379b4 100644
--- a/core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java
+++ b/core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java
@@ -562,7 +562,7 @@ protected void registerMaterializations() {
   void ensureRootConverters() {
     final Set<RelSubset> subsets = new HashSet<>();
     for (RelNode rel : root.getRels()) {
-      if (rel instanceof AbstractConverter) {
+      if (rel instanceof AbstractConverter && !topDownOpt) {
         subsets.add((RelSubset) ((AbstractConverter) rel).getInput());
       }
     }