(doris) 18/30: [fix](nereids)prevent null pointer access if translate expression fails (#33990)

[dataroaring]

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

dataroaring pushed a commit to branch branch-4.0-preview
in repository https://gitbox.apache.org/repos/asf/doris.git

commit c2c95f89dbb2447adbc3e6296c74ad224c854cad
Author: starocean999 <40539150+starocean...@users.noreply.github.com>
AuthorDate: Fri Apr 26 09:26:36 2024 +0800

    [fix](nereids)prevent null pointer access if translate expression fails 
(#33990)
---
 .../doris/nereids/rules/expression/rules/FoldConstantRuleOnBE.java   | 5 +++++
 1 file changed, 5 insertions(+)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/expression/rules/FoldConstantRuleOnBE.java
 
b/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/expression/rules/FoldConstantRuleOnBE.java
index 3c2f0d546dd..e0e19bd19e2 100644
--- 
a/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/expression/rules/FoldConstantRuleOnBE.java
+++ 
b/fe/fe-core/src/main/java/org/apache/doris/nereids/rules/expression/rules/FoldConstantRuleOnBE.java
@@ -215,6 +215,11 @@ public class FoldConstantRuleOnBE implements 
ExpressionPatternRuleFactory {
                 LOG.warn("expression {} translate to legacy expr failed. ", 
expr, e);
                 return;
             }
+            if (staleExpr == null) {
+                // just return, it's a fail-safe
+                LOG.warn("expression {} translate to legacy expr failed. ", 
expr);
+                return;
+            }
             tExprMap.put(id, staleExpr.treeToThrift());
         } else {
             for (int i = 0; i < expr.children().size(); i++) {


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