From d5958f2af7651f5e7e5b2f85cae6be22195ef802 Mon Sep 17 00:00:00 2001 From: starocean999 <12095047@qq.com> Date: Wed, 24 Apr 2024 11:18:19 +0800 Subject: [PATCH] [fix](nereids)prevent null pointer access if translate expression fails --- .../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 e334eb8425ec6c..89955dc4950e62 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 @@ -136,6 +136,11 @@ private void collectConst(Expression expr, Map constMap, Map 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++) {