[jira] [Commented] (DRILL-2884) Have cancel() cause "query canceled" rather than just "ResultSet closed"

["Daniel Barclay (Drill) (JIRA)" <ji...@apache.org>]

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

Daniel Barclay (Drill) commented on DRILL-2884:
-----------------------------------------------

Reviews are at https://reviews.apache.org/r/33651/ and https://reviews.apache.org/r/33652/.

> Have cancel() cause "query canceled" rather than just "ResultSet closed"
> ------------------------------------------------------------------------
>
>                 Key: DRILL-2884
>                 URL: https://issues.apache.org/jira/browse/DRILL-2884
>             Project: Apache Drill
>          Issue Type: Bug
>            Reporter: Daniel Barclay (Drill)
>            Assignee: Daniel Barclay (Drill)
>         Attachments: DRILL-2884.1Core.1.patch.txt, DRILL-2884.2Hygiene.1.patch.txt, DRILL-2884.2Hygiene.2.patch.txt
>
>
> After Statement.cancel() is called for a SQL statement's execution, a subsequent call to a method on the ResultSet for that execution yields a "ResultSet already closed" exception.  (This isn't fully implemented yet, but it is for, e.g., next().)
> A call to cancel() should result in an explicit "execution canceled" exception rather than the usual "already closed" exception (so that it doesn't seem like a Drill programming error rather an expected result of canceling).



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