You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by prasadns14 <gi...@git.apache.org> on 2017/10/12 06:33:12 UTC

[GitHub] drill pull request #988: DRILL-5864: Handled projection of non-existent fiel...

GitHub user prasadns14 opened a pull request:

    https://github.com/apache/drill/pull/988

    DRILL-5864: Handled projection of non-existent field in MapRDB JSON

    When atleast one of the projected fields exists in the MapRDB JSON table, we don't have problem handling the non-existing fields.
    Taking cue from this observation, I included _id field in the projected columns list if it is not already present. This way we don't have to worry about NPE.
    
    @paul-rogers Please review

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/prasadns14/drill DRILL-5864

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/drill/pull/988.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #988
    
----
commit ff89dcf76ff5017c33d486d6db56b16268a8abaa
Author: Prasad Nagaraj Subramanya <pr...@gmail.com>
Date:   2017-10-12T06:25:28Z

    DRILL-5864: Handled projection of non-existent field in MapRDB JSON

----


---

[GitHub] drill pull request #988: DRILL-5864: Handled projection of non-existent fiel...

Posted by prasadns14 <gi...@git.apache.org>.

Github user prasadns14 closed the pull request at:

    https://github.com/apache/drill/pull/988


---