[ 
https://issues.apache.org/jira/browse/TAJO-1574?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14503887#comment-14503887
 ] 

ASF GitHub Bot commented on TAJO-1574:
--------------------------------------

Github user dongjoon-hyun commented on the pull request:

    https://github.com/apache/tajo/pull/548#issuecomment-94587626
  
    Sure. The changed codes are in `tajo-algebra` and `tajo-plan`. I will add a 
testcase to cover both cases in somewhere.


> Fix NPE on natural join
> -----------------------
>
>                 Key: TAJO-1574
>                 URL: https://issues.apache.org/jira/browse/TAJO-1574
>             Project: Tajo
>          Issue Type: Bug
>    Affects Versions: 0.10.0
>            Reporter: Dongjoon Hyun
>            Assignee: Dongjoon Hyun
>            Priority: Minor
>         Attachments: TAJO-1574.Hyun.150421.0.patch.txt
>
>
> NPE occurs in a natural join.
> {code}
> default> create table t(a int);
> default> insert overwrite into t select 1;
> default> create table s(a int);
> default> insert overwrite into s select 1;
> default> select * from t natural join s;
> {code}
> The main reason is *joinQual* is null.
> {code}
>    public RESULT visitJoin(CONTEXT ctx, Stack<Expr> stack, Join expr) throws 
> PlanningException {
>      stack.push(expr);
> -    visit(ctx, stack, expr.getQual());
> +    if (expr.getQual() != null) {
> +      visit(ctx, stack, expr.getQual());
> +    }
> {code}
> {code}
> -    join.joinQual = (Expr) joinQual.clone();
> +    if (joinQual != null) {
> +      join.joinQual = (Expr) joinQual.clone();
> +    }
> {code}
> After fix, we need to get the following result.
> {code}
> default> select * from t natural join s;
> Progress: 100%, response time: 0.293 sec
> a,  a
> -------------------------------
> 1,  1
> (1 rows, 0.293 sec, 4 B selected)
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to