commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Liam Coughlin <lscough...@gmail.com>
Subject Re: [dbutils] bugfixing/java5 branches ready for merge
Date Wed, 25 Feb 2009 15:05:22 GMT
DBUTILS-49        QueryRunner - fillStatement method does not work for
PostgreSQL database drivers<https://issues.apache.org/jira/browse/DBUTILS-49>
I'm pretty sure this is addressed with the fill change to use the
parametermetadata.

DBUTILS-48     Maintaining a parallel Java 1.5 version of
DButils<https://issues.apache.org/jira/browse/DBUTILS-49>
You folks are covering this with the different branch configs post buf fix.

DBUTILS-47     Add a StatementFiller to be able to provide a Bean for
updates, instead of bean
values.<https://issues.apache.org/jira/browse/DBUTILS-49>
I'm pretty sure this is outside the scope of dbutils, and would be covered
by things users of the library add
to it's extensible framework.

DBUTILS-43     an helper set of classes for JDBC processing with prepared
statement <https://issues.apache.org/jira/browse/DBUTILS-49>
This is not trivial -- tackle it now, or tackle it later?

DBUTILS-30     [dbutils] New ResultSetHandler -
ObjectHandler<https://issues.apache.org/jira/browse/DBUTILS-49>
I'm pretty sure this is outside the scope of dbutils, and would be covered
by things users of the library add
to it's extensible framework.

DBUTILS-28     [dbutils] Stored Procedure Runner and Bean Reuse Handler code
submission <https://issues.apache.org/jira/browse/DBUTILS-49>
This is sort of a weird double issue that rolls up DBUTILS-43 and
DBUTILS-30.

I think you probably want to defer any work with stored procedures for a
little while because there's just more interesting and important things to
work on.

In short -- you should probably just close them or mark them inactive.


On Wed, Feb 25, 2009 at 5:07 AM, Henri Yandell <flamefew@gmail.com> wrote:

> On Tue, Feb 24, 2009 at 9:55 AM, Dan Fabulich <dan@fabulich.com> wrote:
> > Henri Yandell wrote:
> >
> >> 742870 - ?? - Lacking Unit Tests, not liking the catch Exception.
> >> RuntimeException throwing needs String arg. Generally not trusting the
> >> Java API here to work beautifully and wanting to have covered a bunch
> >> of use cases.
> >
> > Thanks for reviewing!
> >
> > I tweaked exception handling and added unit tests in bugfixing branch
> > revision 747449.  As I wrote the unit tests I manuallly looked at the
> > exception stacktraces and verified that they provided useful error
> messages
> > to the user. http://svn.apache.org/viewvc?view=rev&revision=747449
> >
> > I also merged those changes to java5 branch revision 747452.
> >
> >> So apart from the one commit, I'm +1 on the changes on the bugfixes
> >> branch.
> >
> > In that case, I think we're ready to contemplate executing the plan I'd
> > identified earlier:
> >
> >>> If I were a commons/dbutils committer right now, I think I'd probably
> do
> >>> these things (all of which require committer karma):
> >>>
> >>> 1) Merge "bugfixing" back to trunk
> >>> 2) Close out all of the bugs I fixed
> >>> 3) Stage/vote on a dbutils-1.2 release based on bugfixing/trunk
> >>> 4) Make a dbutils/1.x branch
> >>> 5) Merge "java5" back to trunk
> >>> 6) Stage/vote on a dbutils-2.0 release based on java5/trunk
> >
> > How do we want to begin work on this?  Henri, do you want to these steps?
> > Alternately, I can do them myself if I get the karma to do so...?
>
> I've gone ahead with the merge and bug closing.
>
> Any thoughts on the remaining 6 open issues in JIRA?
>
> Hen
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message