[VOTE] Release of DbUtils 1.3 RC2

[Dan Fabulich]

This release includes support for Java5 generics and varargs.

RC2 is exactly like RC1, but I updated the site examples and the change
report to emphasize more clearly that this version includes
generics/varargs.

I believe it to be a backwards compatible binary, but I'm still only 99% sure
of this; clirr reports a bunch of compatibility errors that look like nonsense.
I did run the compiled 1.2 tests against 1.3 and it looks OK.

+1 from me!

--

Tag:

https://svn.apache.org/repos/asf/commons/proper/dbutils/tags/DBUTILS_1_3_RC2/

Site:

http://people.apache.org/builds/commons/dbutils/1.3/RC2/site/index.html

Binaries:

http://people.apache.org/builds/commons/dbutils/1.3/RC2/staged/commons-dbutils/commons-dbutils/1.3/

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



---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

[Julien Aymé]

It looks all good !

+1 from me , changes and examples are now corrected :-)

Julien

2009/11/6 Dan Fabulich <[hidden email]>:

>
> This release includes support for Java5 generics and varargs.
>
> RC2 is exactly like RC1, but I updated the site examples and the change
> report to emphasize more clearly that this version includes
> generics/varargs.
>
> I believe it to be a backwards compatible binary, but I'm still only 99%
> sure of this; clirr reports a bunch of compatibility errors that look like
> nonsense. I did run the compiled 1.2 tests against 1.3 and it looks OK.
>
> +1 from me!
>
> --
>
> Tag:
>
> https://svn.apache.org/repos/asf/commons/proper/dbutils/tags/DBUTILS_1_3_RC2/
>
> Site:
>
> http://people.apache.org/builds/commons/dbutils/1.3/RC2/site/index.html
>
> Binaries:
>
> http://people.apache.org/builds/commons/dbutils/1.3/RC2/staged/commons-dbutils/commons-dbutils/1.3/
>
> [ ] +1 release it
> [ ] +0 go ahead I don't care
> [ ] -1 no, do not release it because
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [hidden email]
> For additional commands, e-mail: [hidden email]
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

[Jörg Schaible]

Dan Fabulich wrote:

>
> This release includes support for Java5 generics and varargs.

[snip]

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

Builds fine for my compiler zoo.

- Jörg


---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

[Luc Maisonobe]

Jörg Schaible a écrit :
> Dan Fabulich wrote:
>
>> This release includes support for Java5 generics and varargs.

+0 as there are quite a few checkstyle errors and findbugs errors. These
errors are not blocking, but could be easily fixed. The checkstyle
errors are all missing javadoc. The findbugs errors are mostly
performance issues, one bad practice and two exposure of internal
representations.

Luc

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

[James Carman]

If you don't fix the exposure of internals, it might mess things up
later if you want to change the implementation.  I'd say those should
be fixed.

On Sat, Nov 7, 2009 at 7:41 AM, Luc Maisonobe <[hidden email]> wrote:

> Jörg Schaible a écrit :
>> Dan Fabulich wrote:
>>
>>> This release includes support for Java5 generics and varargs.
>
> +0 as there are quite a few checkstyle errors and findbugs errors. These
> errors are not blocking, but could be easily fixed. The checkstyle
> errors are all missing javadoc. The findbugs errors are mostly
> performance issues, one bad practice and two exposure of internal
> representations.
>
> Luc
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [hidden email]
> For additional commands, e-mail: [hidden email]
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

[Dan Fabulich]

James Carman wrote:

> If you don't fix the exposure of internals, it might mess things up
> later if you want to change the implementation.  I'd say those should
> be fixed.

It's a little late for that... this bug was in version 1.2.

But I fixed it anyway as DBUTILS-63 in revision 833753.  I'll roll out a
new RC.

-Dan

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

[James Carman]

No broken windows, man. :)


On Sat, Nov 7, 2009 at 3:27 PM, Dan Fabulich <[hidden email]> wrote:

> James Carman wrote:
>
>> If you don't fix the exposure of internals, it might mess things up
>> later if you want to change the implementation.  I'd say those should
>> be fixed.
>
> It's a little late for that... this bug was in version 1.2.
>
> But I fixed it anyway as DBUTILS-63 in revision 833753.  I'll roll out a new
> RC.
>
> -Dan
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [hidden email]
> For additional commands, e-mail: [hidden email]
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]