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 21bb7fea07 [minor](Nereids): convert if condition into a check (#20855)
21bb7fea07 is described below
commit 21bb7fea072da382cc949a0d488b66404f666a46
Author: jakevin <[email protected]>
AuthorDate: Thu Jun 15 20:10:56 2023 +0800
[minor](Nereids): convert if condition into a check (#20855)
---
fe/fe-core/src/main/java/org/apache/doris/nereids/memo/Memo.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/memo/Memo.java
b/fe/fe-core/src/main/java/org/apache/doris/nereids/memo/Memo.java
index 04626a2055..e10c39c323 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/nereids/memo/Memo.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/memo/Memo.java
@@ -366,7 +366,8 @@ public class Memo {
throw new IllegalStateException("Insert a plan into targetGroup
but differ in logicalproperties");
}
Optional<GroupExpression> groupExpr = plan.getGroupExpression();
- if (groupExpr.isPresent() &&
groupExpressions.containsKey(groupExpr.get())) {
+ if (groupExpr.isPresent()) {
+
Preconditions.checkState(groupExpressions.containsKey(groupExpr.get()));
return CopyInResult.of(false, groupExpr.get());
}
List<Group> childrenGroups = Lists.newArrayList();
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
