[doris] branch master updated: [fix](Nereids): `FilterOrSelf` shouldn't `And` all predicates.. (#16491)

[jakevin]

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

jakevin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new 98c741d664 [fix](Nereids): `FilterOrSelf` shouldn't `And` all 
predicates.. (#16491)
98c741d664 is described below

commit 98c741d66423508789f6e49c75696ed254c92af3
Author: jakevin <jakevin...@gmail.com>
AuthorDate: Wed Feb 8 12:42:22 2023 +0800

    [fix](Nereids): `FilterOrSelf` shouldn't `And` all predicates.. (#16491)
---
 .../src/main/java/org/apache/doris/nereids/util/PlanUtils.java       | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/nereids/util/PlanUtils.java 
b/fe/fe-core/src/main/java/org/apache/doris/nereids/util/PlanUtils.java
index 36c9d52b28..a5401258cf 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/nereids/util/PlanUtils.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/util/PlanUtils.java
@@ -44,7 +44,10 @@ public class PlanUtils {
     }
 
     public static Optional<LogicalFilter<? extends Plan>> 
filter(Set<Expression> predicates, Plan plan) {
-        return ExpressionUtils.optionalAnd(predicates).map(opt -> new 
LogicalFilter<>(predicates, plan));
+        if (predicates.isEmpty()) {
+            return Optional.empty();
+        }
+        return Optional.of(new LogicalFilter<>(predicates, plan));
     }
 
     public static Plan filterOrSelf(Set<Expression> predicates, Plan plan) {


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