You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by bi...@apache.org on 2017/09/18 05:44:18 UTC
[25/29] kylin git commit: Minor: add FK-PK string array compare in
equals()
Minor: add FK-PK string array compare in equals()
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/cfe5c447
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/cfe5c447
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/cfe5c447
Branch: refs/heads/security_update
Commit: cfe5c447c1daad3d139a7c6d678e603d5683e8d8
Parents: 7640c94
Author: Yifan Zhang <ev...@gmail.com>
Authored: Mon Sep 18 00:06:41 2017 +0800
Committer: Dong Li <li...@apache.org>
Committed: Mon Sep 18 01:00:55 2017 +0800
----------------------------------------------------------------------
.../src/main/java/org/apache/kylin/metadata/model/JoinDesc.java | 5 +++++
1 file changed, 5 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kylin/blob/cfe5c447/core-metadata/src/main/java/org/apache/kylin/metadata/model/JoinDesc.java
----------------------------------------------------------------------
diff --git a/core-metadata/src/main/java/org/apache/kylin/metadata/model/JoinDesc.java b/core-metadata/src/main/java/org/apache/kylin/metadata/model/JoinDesc.java
index c8a5110..b16ad7d 100644
--- a/core-metadata/src/main/java/org/apache/kylin/metadata/model/JoinDesc.java
+++ b/core-metadata/src/main/java/org/apache/kylin/metadata/model/JoinDesc.java
@@ -171,7 +171,12 @@ public class JoinDesc implements Serializable {
return false;
JoinDesc other = (JoinDesc) obj;
+
// note pk/fk are sorted, sortByFK()
+ if (!Arrays.equals(foreignKey, other.foreignKey))
+ return false;
+ if (!Arrays.equals(primaryKey, other.primaryKey))
+ return false;
if (!Arrays.equals(foreignKeyColumns, other.foreignKeyColumns))
return false;
if (!Arrays.equals(primaryKeyColumns, other.primaryKeyColumns))