ASF GitHub Bot commented on DRILL-4950:

GitHub user sudheeshkatkam opened a pull request:


    DRILL-4950: Remove incorrect false condition; consume all empty batches

    @parthchandra please review

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

    $ git pull https://github.com/sudheeshkatkam/drill DRILL-4950

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


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

    This closes #621
commit f349648b0782aba80def7f549b6b57ad32ad3881
Author: Sudheesh Katkam <skat...@maprtech.com>
Date:   2016-10-18T21:22:53Z

    DRILL-4950: Remove incorrect false condition; consume all empty batches


> Consume Spurious Empty Batches in JDBC
> --------------------------------------
>                 Key: DRILL-4950
>                 URL: https://issues.apache.org/jira/browse/DRILL-4950
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Client - JDBC
>            Reporter: Sudheesh Katkam
>            Assignee: Sudheesh Katkam
>            Priority: Blocker
>             Fix For: 1.9.0
> In 
> [DrillCursor|https://github.com/apache/drill/blob/master/exec/jdbc/src/main/java/org/apache/drill/jdbc/impl/DrillCursor.java#L199],
>  consume all empty batches, not just non-continuous empty batches. This 
> results in query cancellation (from sqlline) and incomplete results.
> Introduced (regression?) in DRILL-2548.

This message was sent by Atlassian JIRA

Reply via email to