You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by ja...@apache.org on 2023/06/15 12:11:03 UTC
[doris] branch master updated: [minor](Nereids): convert if condition into a check (#20855)
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 <ja...@gmail.com>
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: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org