drill-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (DRILL-2961) Statement.setQueryTimeout() should throw a SQLException
Date Wed, 06 May 2015 16:40:02 GMT

    [ 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 <kkhatua@maprtech.com>
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)

Mime
View raw message