drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From arina-ielchiieva <...@git.apache.org>
Subject [GitHub] drill pull request #714: DRILL-4919: Fix select count(1) / count(*) on csv w...
Date Thu, 29 Dec 2016 17:02:52 GMT
GitHub user arina-ielchiieva opened a pull request:

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

    DRILL-4919: Fix select count(1) / count(*) on csv with header

    When columns from file are not needed in result (`isSkipQuery = true`), empty list of
columns is replaced for text files with `columns[0]`. But currently text files with headers
do not support `columns` syntax, so for these type of files we need to replace empty list
of columns with default named column.

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

    $ git pull https://github.com/arina-ielchiieva/drill DRILL-4919

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

    https://github.com/apache/drill/pull/714.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 #714
    
----
commit c2d1e222fac64dc6c5b177c983a2e5b2d5227190
Author: Arina Ielchiieva <arina.yelchiyeva@gmail.com>
Date:   2016-12-29T15:42:53Z

    DRILL-4919: Fix select count(1) / count(*) on csv with header

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message