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 <[email protected]>
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: [email protected]
For additional commands, e-mail: [email protected]
