[
https://issues.apache.org/jira/browse/DRILL-2961?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14530845#comment-14530845
]
ASF GitHub Bot commented on DRILL-2961:
---------------------------------------
GitHub user kkhatua opened a pull request:
https://github.com/apache/drill/pull/71
DRILL-2961: Throw SQLException when attempting to set query timeout
Statement.setQueryTimeout(int seconds) is a No-Op, leading the user to
believe that a timeout has been set. This provides the correct response
(SQLException for 'method not supported')
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/kkhatua/drill master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/drill/pull/71.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 #71
----
commit 1d3ffe84f3237d614b524702e89be8cce1509931
Author: Kunal Khatua <[email protected]>
Date: 2015-05-06T16:37:56Z
DRILL-2961: Throw SQLException when attempting to set query timeout
Statement.setQueryTimeout(int seconds) is a No-Op, leading the user to
believe that a timeout has been set. This provides the correct response
(SQLException for 'method not supported')
----
> Statement.setQueryTimeout() should throw a SQLException
> -------------------------------------------------------
>
> Key: DRILL-2961
> URL: https://issues.apache.org/jira/browse/DRILL-2961
> Project: Apache Drill
> Issue Type: Bug
> Components: Client - JDBC
> Affects Versions: 1.0.0
> Environment: RHEL 6.4
> Reporter: Kunal Khatua
> Assignee: Daniel Barclay (Drill)
> Fix For: 1.0.0
>
>
> When trying to set the timeout for a Drill Statement object, Drill does not
> report any SQLException which makes the developer incorrectly believe that a
> timeout has been set.
> The operation should throw the exception:
> java.sql.SQLException: Method not supported
> at
> org.apache.drill.jdbc.DrillStatement.setQueryTimeout(DrillStatement.java)
> ....
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)