You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by li...@apache.org on 2016/12/08 06:48:18 UTC

kylin git commit: minor but important, ColumnDesc.equals() check only table and name

Repository: kylin
Updated Branches:
  refs/heads/master 4c39fcb0e -> 8b6462063


minor but important, ColumnDesc.equals() check only table and name


Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/8b646206
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/8b646206
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/8b646206

Branch: refs/heads/master
Commit: 8b64620631200a724117dec97e3c69c19d9cf22c
Parents: 4c39fcb
Author: Li Yang <li...@apache.org>
Authored: Thu Dec 8 14:48:11 2016 +0800
Committer: Li Yang <li...@apache.org>
Committed: Thu Dec 8 14:48:11 2016 +0800

----------------------------------------------------------------------
 .../main/java/org/apache/kylin/metadata/model/ColumnDesc.java  | 6 ------
 1 file changed, 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/8b646206/core-metadata/src/main/java/org/apache/kylin/metadata/model/ColumnDesc.java
----------------------------------------------------------------------
diff --git a/core-metadata/src/main/java/org/apache/kylin/metadata/model/ColumnDesc.java b/core-metadata/src/main/java/org/apache/kylin/metadata/model/ColumnDesc.java
index 403eaaf..2da1f5e 100644
--- a/core-metadata/src/main/java/org/apache/kylin/metadata/model/ColumnDesc.java
+++ b/core-metadata/src/main/java/org/apache/kylin/metadata/model/ColumnDesc.java
@@ -197,12 +197,6 @@ public class ColumnDesc implements Serializable {
         } else if (!table.getIdentity().equals(other.table.getIdentity()))
             return false;
         
-        if (datatype == null) {
-            if (other.datatype != null)
-                return false;
-        } else if (!datatype.equals(other.datatype))
-            return false;
-
         return true;
     }