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

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

Notes re (current) decision to have successful cancelation cause exception to 
the thrown from execute...(...) or next(...) (etc.):

[Some Oracle 
documentation|http://docs.oracle.com/cd/B14117_01/java.101/b10979/tips.htm#BACDAICJ]
 says:

bq.  "The JDBC standard method Statement.cancel() attempts to cleanly stop the 
execution of a SQL statement by sending a message to the database. In response, 
the database stops execution and replies with an error message. "

> 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)
>             Fix For: 1.0.0
>
>         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)

Reply via email to