[doris] branch master updated: [minor](Nereids): convert if condition into a check (#20855)

[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 21bb7fea07 [minor](Nereids): convert if condition into a check (#20855)
21bb7fea07 is described below

commit 21bb7fea072da382cc949a0d488b66404f666a46
Author: jakevin <jakevin...@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-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org