From 88a3d08fee9d9239ad8b5969dae22262185fdeba Mon Sep 17 00:00:00 2001 From: GoGoWen <82132356+GoGoWen@users.noreply.github.com> Date: Sun, 16 Jan 2022 10:42:24 +0800 Subject: [PATCH] [fix] fix NPE in SysVariableDesc::equal (#7766) --- .../main/java/org/apache/doris/analysis/SysVariableDesc.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/fe/fe-core/src/main/java/org/apache/doris/analysis/SysVariableDesc.java b/fe/fe-core/src/main/java/org/apache/doris/analysis/SysVariableDesc.java index 0ade37866a..29e6a3c18d 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/analysis/SysVariableDesc.java +++ b/fe/fe-core/src/main/java/org/apache/doris/analysis/SysVariableDesc.java @@ -32,6 +32,8 @@ import org.apache.doris.thrift.TStringLiteral; import com.google.common.base.Strings; +import java.util.Objects; + // System variable // Converted to StringLiteral in analyze, if this variable is not exist, throw AnalysisException. public class SysVariableDesc extends Expr { @@ -189,6 +191,7 @@ public class SysVariableDesc extends Expr { if (!setType.equals(((SysVariableDesc) obj).getSetType())) { return false; } - return literalExpr.equals(((SysVariableDesc) obj).getLiteralExpr()); + + return Objects.equals(literalExpr, ((SysVariableDesc) obj).getLiteralExpr()); } }