Re: setQueryTimeout problem !?!?!

From: Oliver Jowett <oliver(at)opencloud(dot)com>
To: robert engels <rengels(at)ix(dot)netcom(dot)com>
Cc: pgsql-jdbc(at)postgresql(dot)org
Subject: Re: setQueryTimeout problem !?!?!
Date: 2008-03-18 05:57:45
Message-ID: 47DF59D9.40502@opencloud.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-jdbc

robert engels wrote:
> Here is the javadoc. The method explicitly states when it throws an
> exception:
>
> /**
> * Sets the number of seconds the driver will wait for a
> * <code>Statement</code> object to execute to the given number of
> seconds.
> * If the limit is exceeded, an <code>SQLException</code> is thrown.
> *
> * @param seconds the new query timeout limit in seconds; zero means
> * there is no limit
> * @exception SQLException if a database access error occurs
> * or the condition seconds >= 0 is not satisfied
> * @see #getQueryTimeout
> */
>
> Even if this is not sufficient, it worked this way in the previous
> driver versions - so I guess there should be evidence the other way in
> order to change it.
>
> This is not a problem when using the 8.3 driver against an 8.3 database
> - only seems to happen with a 8.1 db (maybe 8.2???)

I did read the javadoc .. It does not say that the driver should
silently do nothing if query timeouts are unsupported.

The Java 6 javadoc is somewhat more explicit:

> Sets the number of seconds the driver will wait for a Statement object to execute to the given number of seconds. If the limit is exceeded, an SQLException is thrown. A JDBC driver must apply this limit to the execute, executeQuery and executeUpdate methods. JDBC driver implementations may also apply this limit to ResultSet methods (consult your driver vendor documentation for details).

Drivers "must apply this limit", but we cannot currently do that.

The argument here is that asking the driver to do something it can't
actually do is a "database access error". So you get a SQLException if
you ask for a non-zero timeout, since that is not currently supported.

See previous discussion that prompted this change at
http://archives.postgresql.org/pgsql-jdbc/2008-01/msg00088.php

-O

In response to

Responses

Browse pgsql-jdbc by date

  From Date Subject
Next Message Dave Cramer 2008-03-18 11:53:09 Re: setQueryTimeout problem !?!?!
Previous Message Kris Jurka 2008-03-18 05:50:59 Re: JDBC rewriting a bad query?