You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by sh...@apache.org on 2020/04/28 08:38:31 UTC

[kylin] branch master updated: minor, if o == null, o instanceof Array will return false

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

shaofengshi pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kylin.git


The following commit(s) were added to refs/heads/master by this push:
     new 346b53c  minor, if o == null, o instanceof Array will return false
346b53c is described below

commit 346b53c705ef48394a53faed1ff1efced7c808e8
Author: leo <18...@163.com>
AuthorDate: Thu Apr 23 22:50:53 2020 +0800

    minor, if o == null, o instanceof Array will return false
---
 core-common/src/main/java/org/apache/kylin/common/util/Array.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/core-common/src/main/java/org/apache/kylin/common/util/Array.java b/core-common/src/main/java/org/apache/kylin/common/util/Array.java
index b25b764..4b32f6d 100644
--- a/core-common/src/main/java/org/apache/kylin/common/util/Array.java
+++ b/core-common/src/main/java/org/apache/kylin/common/util/Array.java
@@ -36,7 +36,7 @@ public class Array<T> implements Comparable<Array<T>>, java.io.Serializable {
 
     @Override
     public boolean equals(Object o) {
-        if (o != null && o instanceof Array) {
+        if (o instanceof Array) {
             return Arrays.equals(this.data, ((Array<?>) o).data);
         }
         return false;