You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tajo.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2014/07/15 07:58:05 UTC

[jira] [Commented] (TAJO-904) ORDER BY Null first support

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

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

Github user babokim commented on the pull request:

    https://github.com/apache/tajo/pull/70#issuecomment-48993442
  
    Hyunsik, I agree your opinion. Please go ahead


> ORDER BY Null first support
> ---------------------------
>
>                 Key: TAJO-904
>                 URL: https://issues.apache.org/jira/browse/TAJO-904
>             Project: Tajo
>          Issue Type: Bug
>            Reporter: Hyoungjun Kim
>            Assignee: Hyoungjun Kim
>            Priority: Trivial
>
> ORDER BY with a null column miss some data. I found RowStoreEncoder has a bug dealing with NULL values. 
> "continue" statement should be add.
> {code}
>       Column col;
>       for (int i = 0; i < schema.size(); i++) {
>         if (tuple.isNull(i)) {
>           nullFlags.set(i);
>           continue;
>         }
>         col = schema.getColumn(i);
> {code}



--
This message was sent by Atlassian JIRA
(v6.2#6252)