KYLIN-2608 Bubble sort bug in JoinDesc

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

Branch: refs/heads/KYLIN-2624
Commit: a059064df85d26014150709273cb2535805adadf
Parents: f277b2d
Author: Hongbin Ma <mahong...@apache.org>
Authored: Thu May 11 16:24:56 2017 +0800
Committer: liyang-gmt8 <liy...@apache.org>
Committed: Thu May 11 16:27:38 2017 +0800

----------------------------------------------------------------------
 .../src/main/java/org/apache/kylin/metadata/model/JoinDesc.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/a059064d/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 dd1500b..eb82ace 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
@@ -125,7 +125,7 @@ public class JoinDesc implements Serializable {
         int n = foreignKey.length;
         for (int i = 0; i < n - 1 && cont; i++) {
             cont = false;
-            for (int j = i; j < n - 1; j++) {
+            for (int j = 0; j < n - 1 - i; j++) {
                 int jj = j + 1;
                 if (foreignKey[j].compareTo(foreignKey[jj]) > 0) {
                     swap(foreignKey, j, jj);

Reply via email to