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

morningman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-doris.git


The following commit(s) were added to refs/heads/master by this push:
     new 88a3d08  [fix] fix NPE in SysVariableDesc::equal (#7766)
88a3d08 is described below

commit 88a3d08fee9d9239ad8b5969dae22262185fdeba
Author: GoGoWen <[email protected]>
AuthorDate: Sun Jan 16 10:42:24 2022 +0800

    [fix] fix NPE in SysVariableDesc::equal (#7766)
---
 .../src/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 0ade378..29e6a3c 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());
     }
 }

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to