[kylin] 02/10: minor, fix query handling right outer join case

[li...@apache.org]

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 b09352c4701a42cd83e6a5cbb9125ee8b7ddc0a5
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.