commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebb <>
Subject Re: [VOTE] Release of DbUtils 1.2 RC1
Date Mon, 09 Mar 2009 03:14:26 GMT
On 08/03/2009, Dan Fabulich <> wrote:
>  My first attempt at releasing a commons project; please be gentle. :-)
>  Compatibility warning: This version is mostly a bugfix release, but to fix
> DBUTILS-31 we had to upgrade the JVM dependency from JDK 1.3 to JDK 1.4.
> Except for that, it is backwards compatible with DbUtils 1.1.
>  Although this project has reasonable unit tests, it has no integration
> tests with any actual databases; it is quite possible that the fix for
> DBUTILS-31 has broken something on Oracle, MS SQL Server, Derby, or your
> favorite database.

The Unit test


looks rather odd, in that it assigns and checks the row[] array to be
non-null within the iterator loop, but checks row[0] etc outside the
loop. Is it really intended to only check the last row[] array? If so,
it ought to check for null first.

Similar comments apply to:


Also,  ProxyFactoryTest uses "instanceof" checks against methods that
are declared to return the class being tested - these should surely
check for null instead?

>  To verify DBUTILS-31, use QueryRunner to put a null value in a field, e.g.
> with QueryRunner.update.  Ideally it would be good to verify putting nulls
> in fields of various types: char, varchar, int, boolean, date, etc.
>  --
>  Tag:

DOAP ought to have an AL header.

>  Site:

"...and relies only on a standard Java 1.3 or later JRE. "

That should be 1.4.

Given that it is now using 1.4, Junit could be updated to 3.8.2

>  Binaries:

Sigs and hashes OK
N&L files look OK.

Source archive agrees with SVN tag.

>  [ ] +1 release it
>  [ ] +0 go ahead I don't care
>  [ ] -1 no, do not release it because

-0.5 because the unit tests seem wrong.

> ---------------------------------------------------------------------
>  To unsubscribe, e-mail:
>  For additional commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message