It's been nearly a year since 1.0 was released. I'll work on getting DbUtils 1.1 released relatively soon.
David --- Alik Eliashberg <[EMAIL PROTECTED]> wrote: > Ah, got it! Do you know when is the next release of DbUtils (that would > incorporate this fix)? > > Thank you, > > Alik > > David Graham wrote: > > >This was fixed some time ago as you can see in the latest source here: > >http://jakarta.apache.org/commons/dbutils/xref/org/apache/commons/dbutils/QueryRunner.html#235 > > > >The fix is in the nightly builds. > > > >Thanks, > >David > > > > > >--- Alik Eliashberg <[EMAIL PROTECTED]> wrote: > > > > > > > >>Hello, > >> > >>I just started using the DbUtils package. I was reading the source > code > >>and I noticed the following potential problem: > >> > >>QueryRunner has a 'finally' clause in'query' method. This clause > closes > >>both ResultSet and the Statement. The problem is that it uses 'close' > >>not 'closeQuietly'. So it is possible that if the ResultSet close > fails, > >> > >>the Statement will remain open forever causing a driver/db resource > >>leak. Shouldn't the code be rewritten with another nested > 'try/finally': > >> try { > >> ... > >> } finally { > >> try { > >> DbUtils.close(rs); > >> } finally { > >> DbUtils.close(stmt); > >> } > >> } > >>[Alternatively, 'closeQuetly' can be used, but that would suppress the > > >>error...] > >> > >>TIA, > >> > >>Alik > >> > >> > >>--------------------------------------------------------------------- > >>To unsubscribe, e-mail: [EMAIL PROTECTED] > >>For additional commands, e-mail: [EMAIL PROTECTED] > >> > >> > >> > >> > > > > > > > > > >__________________________________ > >Do you Yahoo!? > >Check out the new Yahoo! Front Page. > >www.yahoo.com > > > > > > > >--------------------------------------------------------------------- > >To unsubscribe, e-mail: [EMAIL PROTECTED] > >For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > __________________________________ Do you Yahoo!? Check out the new Yahoo! Front Page. www.yahoo.com --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
