You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by xx...@apache.org on 2023/05/06 06:59:03 UTC

[kylin] 02/38: KYLIN-5521 fix LeftOrInner Join

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

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

commit 54940c9a9dced70a5d64fad376acdccc519b7400
Author: Jiale He <ji...@gmail.com>
AuthorDate: Fri Feb 17 16:11:38 2023 +0800

    KYLIN-5521 fix LeftOrInner Join
---
 .../src/main/java/org/apache/kylin/metadata/model/graph/JoinsGraph.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/core-metadata/src/main/java/org/apache/kylin/metadata/model/graph/JoinsGraph.java b/src/core-metadata/src/main/java/org/apache/kylin/metadata/model/graph/JoinsGraph.java
index b6342fd476..67f8e26bfc 100644
--- a/src/core-metadata/src/main/java/org/apache/kylin/metadata/model/graph/JoinsGraph.java
+++ b/src/core-metadata/src/main/java/org/apache/kylin/metadata/model/graph/JoinsGraph.java
@@ -355,7 +355,7 @@ public class JoinsGraph implements Serializable {
                 }
                 for (Edge targetEdge : edgeList) {
                     if (!edge.equals(targetEdge) && fkSide.equals(targetEdge.pkSide())
-                            && !targetEdge.isLeftOrInnerJoin() && targetEdge.isLeftJoin()) {
+                            && !targetEdge.isLeftOrInnerJoin()) {
                         setJoinToLeftOrInner(targetEdge.join);
                         normalize();
                     }