(doris) 01/09: [minor](nereids): remove useless code (#35325)

[yiguolei]

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

yiguolei pushed a commit to branch branch-2.1
in repository https://gitbox.apache.org/repos/asf/doris.git

commit 41c3a27bce654f8c4cc1ec7bf39cae192d191335
Author: jakevin <jakevin...@gmail.com>
AuthorDate: Fri May 24 16:33:55 2024 +0800

    [minor](nereids): remove useless code (#35325)
---
 .../nereids/rules/rewrite/AdjustConjunctsReturnType.java   |  5 -----
 .../apache/doris/nereids/rules/rewrite/OrExpansion.java    | 14 --------------
 2 files changed, 19 deletions(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/AdjustConjunctsReturnType.java
 
b/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/AdjustConjunctsReturnType.java
index ebc67e7c515..fe5b7ae5780 100644
--- 
a/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/AdjustConjunctsReturnType.java
+++ 
b/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/AdjustConjunctsReturnType.java
@@ -43,11 +43,6 @@ public class AdjustConjunctsReturnType extends 
DefaultPlanRewriter<Void> impleme
         return plan.accept(this, null);
     }
 
-    @Override
-    public Plan visit(Plan plan, Void context) {
-        return super.visit(plan, context);
-    }
-
     @Override
     public Plan visitLogicalFilter(LogicalFilter<? extends Plan> filter, Void 
context) {
         filter = (LogicalFilter<? extends Plan>) super.visit(filter, context);
diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/OrExpansion.java
 
b/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/OrExpansion.java
index 61a90e3aea4..f4bdde0730b 100644
--- 
a/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/OrExpansion.java
+++ 
b/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/rewrite/OrExpansion.java
@@ -90,20 +90,6 @@ public class OrExpansion extends 
DefaultPlanRewriter<OrExpandsionContext> implem
         return plan;
     }
 
-    @Override
-    public Plan visit(Plan plan, OrExpandsionContext ctx) {
-        List<Plan> newChildren = new ArrayList<>();
-        boolean hasNewChildren = false;
-        for (Plan child : plan.children()) {
-            Plan newChild = child.accept(this, ctx);
-            if (newChild != child) {
-                hasNewChildren = true;
-            }
-            newChildren.add(newChild);
-        }
-        return hasNewChildren ? plan.withChildren(newChildren) : plan;
-    }
-
     @Override
     public Plan visitLogicalCTEAnchor(
             LogicalCTEAnchor<? extends Plan, ? extends Plan> anchor, 
OrExpandsionContext ctx) {


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org