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 2018/03/24 12:48:14 UTC
[kylin] 02/02: minor, fix query handling right outer join case
This is an automated email from the ASF dual-hosted git repository.
liyang pushed a commit to branch sync
in repository https://gitbox.apache.org/repos/asf/kylin.git
commit f50120a6543cb3e0a6df84cda183d3ac5e1bcfe5
Author: Sheng Zhang <sh...@kyligence.io>
AuthorDate: Tue Mar 20 15:44:31 2018 +0800
minor, fix query handling right outer join case
If the JoinRelType is right outer join or full join, assign it RIGHT or FULL rather than NULL to avoid NullPointerExcption.
---
query/src/main/java/org/apache/kylin/query/relnode/OLAPJoinRel.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/query/src/main/java/org/apache/kylin/query/relnode/OLAPJoinRel.java b/query/src/main/java/org/apache/kylin/query/relnode/OLAPJoinRel.java
index 304f22b..4182453 100644
--- a/query/src/main/java/org/apache/kylin/query/relnode/OLAPJoinRel.java
+++ b/query/src/main/java/org/apache/kylin/query/relnode/OLAPJoinRel.java
@@ -186,7 +186,7 @@ public class OLAPJoinRel extends EnumerableJoin implements OLAPRel {
JoinRelType joinRelType = this.getJoinType();
String joinType = joinRelType == JoinRelType.INNER ? "INNER"
- : joinRelType == JoinRelType.LEFT ? "LEFT" : null;
+ : joinRelType == JoinRelType.LEFT ? "LEFT" : joinRelType == JoinRelType.RIGHT ? "RIGHT" : "FULL";
join.setType(joinType);
this.context.joins.add(join);
--
To stop receiving notification emails like this one, please contact
liyang@apache.org.