Menu
▾
▴
springframework-developer
|
From: Thomas R. <tho...@tr...> - 2005-03-20 18:24:17
|
I have moved the JDBC RowSet support over from the Sandbox. I will be
adding more JavaDocs and some tests later today - just wanted to get
the initial support committed.
Here is a brief usage sample:
SqlRowSet rset = getJdbcTemplate().queryForRowSet(
"select id, name from mytest where id > ? and name < ?",
new Object[] {new Long(1), "Z"});
int i = 0;
while (rset.next()) {
i++;
System.out.println("Row" + i + " " + rset.getString(2));
}
Thomas
|
|
From: Rod J. <ro...@in...> - 2005-03-21 10:24:15
|
Excellent. This is definitely a good feature to have, particularly for
web apps that are just "window on data" style, where domain objects
don't add a great deal of value.
Thomas Risberg wrote:
> I have moved the JDBC RowSet support over from the Sandbox. I will be
> adding more JavaDocs and some tests later today - just wanted to get the
> initial support committed.
>
> Here is a brief usage sample:
>
> SqlRowSet rset = getJdbcTemplate().queryForRowSet(
> "select id, name from mytest where id > ? and name < ?",
> new Object[] {new Long(1), "Z"});
> int i = 0;
> while (rset.next()) {
> i++;
> System.out.println("Row" + i + " " + rset.getString(2));
> }
>
> Thomas
>
>
>
>
> -------------------------------------------------------
> SF email is sponsored by - The IT Product Guide
> Read honest & candid reviews on hundreds of IT Products from real users.
> Discover which products truly live up to the hype. Start reading now.
> http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
> _______________________________________________
> Springframework-developer mailing list
> Spr...@li...
> https://lists.sourceforge.net/lists/listinfo/springframework-developer
>
--
Rod Johnson
Interface21 - Spring Services from the Source
http://www.springframework.com
Founder, Spring Framework:
http://www.springframework.org
Author, "Expert One-on-One J2EE Development Without EJB"
(May 2004, with Juergen Hoeller).
http://www.amazon.com/exec/obidos/ASIN/0764558315/
Author, "Expert One-on-One J2EE Design and Development"
(October 2002).
http://www.amazon.com/exec/obidos/tg/detail/-/0764543857/
____________________________________________________
Interface21 Limited
Registered Office Summit House, 2-2a Highfield Road, Dartford, Kent DA1 2JY
Registered in England and Wales No. 5187766
____________________________________________________
|
|
From: Juergen H. <ju...@in...> - 2005-03-21 20:06:35
|
Thomas, everybody,
I've just started reviewing this. Looks good!
The only basic thing that needs to change is the location of the SqlRowSet
classes: It currently introduces a circular package dependency between
"jdbc.core" and "jdbc.core.support". The support package should not contain
classes that are directly referenced from "jdbc.core", but this is
unfortunately the case here.
I'll move the SqlRowSet classes to a package "jdbc.support.rowset". I guess
this is worth it, as we have 4 classes there, and the row set stuff itself
is actually pretty independent from JdbcTemplate in the first place.
Additionally, I'll also factor the creation of a CachedRowSet instance out
into a method that can easily be overridden. CachedRowSetImpl, which we're
using by default, is a "com.sun" class, which I feel a little bit
uncomfortable with.
I'd also like to ask for a feature freeze for 1.2 RC1. We're already late; I
intend to finally release RC1 on Wednesday.
Refining CannotAcquireLockException and co is still fine with me, but please
keep me up to date on what's happening there. If someone reworks
documentation or the like, that's fine too, of course - but please refrain
from adding new features, even from trivial (or trivial-looking) ones.
Juergen
-----Original Message-----
From: spr...@li...
[mailto:spr...@li...]On Behalf
Of Thomas Risberg
Sent: Sunday, March 20, 2005 7:24 PM
To: spr...@li...
Subject: [Springframework-developer] Added support for JDBC RowSets
I have moved the JDBC RowSet support over from the Sandbox. I will be
adding more JavaDocs and some tests later today - just wanted to get
the initial support committed.
Here is a brief usage sample:
SqlRowSet rset = getJdbcTemplate().queryForRowSet(
"select id, name from mytest where id > ? and name < ?",
new Object[] {new Long(1), "Z"});
int i = 0;
while (rset.next()) {
i++;
System.out.println("Row" + i + " " + rset.getString(2));
}
Thomas
-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
Springframework-developer mailing list
Spr...@li...
https://lists.sourceforge.net/lists/listinfo/springframework-developer
|
|
From: Thomas R. <tho...@tr...> - 2005-03-22 03:09:57
|
Moving the RowSet classes to a separate package makes sense. Also
factoring out the creation of the CachedRowSet instance is good - I was
planning on providing some kind of pluggable support for different
implementations - might be important in the future as vendors provide
their own compatible implementations. I'm not too crazy about he
com.sun.rowset.CachedRowSetImpl dependency either, but it is part of
the RI and provided in rowset.jar and J2SE 5.0. I added some more
JavaDocs as well.
I have also added the PessimisticLockingException and added translation
slots for it and cannotSerializeTransaction and DedlockLoserDataAccess
as well. We still need to provide error codes to map to these
exceptions.
Thomas
On Mar 21, 2005, at 3:06 PM, Juergen Hoeller wrote:
> Thomas, everybody,
>
> I've just started reviewing this. Looks good!
>
> The only basic thing that needs to change is the location of the
> SqlRowSet
> classes: It currently introduces a circular package dependency between
> "jdbc.core" and "jdbc.core.support". The support package should not
> contain
> classes that are directly referenced from "jdbc.core", but this is
> unfortunately the case here.
>
> I'll move the SqlRowSet classes to a package "jdbc.support.rowset". I
> guess
> this is worth it, as we have 4 classes there, and the row set stuff
> itself
> is actually pretty independent from JdbcTemplate in the first place.
>
> Additionally, I'll also factor the creation of a CachedRowSet instance
> out
> into a method that can easily be overridden. CachedRowSetImpl, which
> we're
> using by default, is a "com.sun" class, which I feel a little bit
> uncomfortable with.
>
> I'd also like to ask for a feature freeze for 1.2 RC1. We're already
> late; I
> intend to finally release RC1 on Wednesday.
>
> Refining CannotAcquireLockException and co is still fine with me, but
> please
> keep me up to date on what's happening there. If someone reworks
> documentation or the like, that's fine too, of course - but please
> refrain
> from adding new features, even from trivial (or trivial-looking) ones.
>
> Juergen
>
>
>
> -----Original Message-----
> From: spr...@li...
> [mailto:spr...@li...]On Behalf
> Of Thomas Risberg
> Sent: Sunday, March 20, 2005 7:24 PM
> To: spr...@li...
> Subject: [Springframework-developer] Added support for JDBC RowSets
>
>
> I have moved the JDBC RowSet support over from the Sandbox. I will be
> adding more JavaDocs and some tests later today - just wanted to get
> the initial support committed.
>
> Here is a brief usage sample:
>
> SqlRowSet rset = getJdbcTemplate().queryForRowSet(
> "select id, name from mytest where id > ? and name <
> ?",
> new Object[] {new Long(1), "Z"});
> int i = 0;
> while (rset.next()) {
> i++;
> System.out.println("Row" + i + " " + rset.getString(2));
> }
>
> Thomas
>
>
>
>
> -------------------------------------------------------
> SF email is sponsored by - The IT Product Guide
> Read honest & candid reviews on hundreds of IT Products from real
> users.
> Discover which products truly live up to the hype. Start reading now.
> http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
> _______________________________________________
> Springframework-developer mailing list
> Spr...@li...
> https://lists.sourceforge.net/lists/listinfo/springframework-developer
>
>
>
> -------------------------------------------------------
> SF email is sponsored by - The IT Product Guide
> Read honest & candid reviews on hundreds of IT Products from real
> users.
> Discover which products truly live up to the hype. Start reading now.
> http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
> _______________________________________________
> Springframework-developer mailing list
> Spr...@li...
> https://lists.sourceforge.net/lists/listinfo/springframework-developer
>
>
|
|
From: Juergen H. <ju...@in...> - 2005-03-22 06:28:38
|
Thomas,
I've already committed all changes yesterday night. The CachedRowSet
creation is currently just factored out into a protected "SqlRowSet
createRowSet" method, where the default implementation creates a Spring
SqlRowSetImpl that wraps a Sun CachedRowSetImpl.
OK, so the Sun RowSet impls are part of JDK 1.5 too, not just of the
separate rowset.jar. I guess that's good enough, so let's not worry about it
too much. The dependency just kicks in when actually requesting disconneced
row sets anyway.
I'll have a look at the refined exception hierarchy today.
Juergen
-----Original Message-----
From: spr...@li...
[mailto:spr...@li...]On Behalf
Of Thomas Risberg
Sent: Tuesday, March 22, 2005 4:10 AM
To: spr...@li...
Subject: Re: [Springframework-developer] Added support for JDBC RowSets
Moving the RowSet classes to a separate package makes sense. Also
factoring out the creation of the CachedRowSet instance is good - I was
planning on providing some kind of pluggable support for different
implementations - might be important in the future as vendors provide
their own compatible implementations. I'm not too crazy about he
com.sun.rowset.CachedRowSetImpl dependency either, but it is part of
the RI and provided in rowset.jar and J2SE 5.0. I added some more
JavaDocs as well.
I have also added the PessimisticLockingException and added translation
slots for it and cannotSerializeTransaction and DedlockLoserDataAccess
as well. We still need to provide error codes to map to these
exceptions.
Thomas
On Mar 21, 2005, at 3:06 PM, Juergen Hoeller wrote:
> Thomas, everybody,
>
> I've just started reviewing this. Looks good!
>
> The only basic thing that needs to change is the location of the
> SqlRowSet
> classes: It currently introduces a circular package dependency between
> "jdbc.core" and "jdbc.core.support". The support package should not
> contain
> classes that are directly referenced from "jdbc.core", but this is
> unfortunately the case here.
>
> I'll move the SqlRowSet classes to a package "jdbc.support.rowset". I
> guess
> this is worth it, as we have 4 classes there, and the row set stuff
> itself
> is actually pretty independent from JdbcTemplate in the first place.
>
> Additionally, I'll also factor the creation of a CachedRowSet instance
> out
> into a method that can easily be overridden. CachedRowSetImpl, which
> we're
> using by default, is a "com.sun" class, which I feel a little bit
> uncomfortable with.
>
> I'd also like to ask for a feature freeze for 1.2 RC1. We're already
> late; I
> intend to finally release RC1 on Wednesday.
>
> Refining CannotAcquireLockException and co is still fine with me, but
> please
> keep me up to date on what's happening there. If someone reworks
> documentation or the like, that's fine too, of course - but please
> refrain
> from adding new features, even from trivial (or trivial-looking) ones.
>
> Juergen
>
>
>
> -----Original Message-----
> From: spr...@li...
> [mailto:spr...@li...]On Behalf
> Of Thomas Risberg
> Sent: Sunday, March 20, 2005 7:24 PM
> To: spr...@li...
> Subject: [Springframework-developer] Added support for JDBC RowSets
>
>
> I have moved the JDBC RowSet support over from the Sandbox. I will be
> adding more JavaDocs and some tests later today - just wanted to get
> the initial support committed.
>
> Here is a brief usage sample:
>
> SqlRowSet rset = getJdbcTemplate().queryForRowSet(
> "select id, name from mytest where id > ? and name <
> ?",
> new Object[] {new Long(1), "Z"});
> int i = 0;
> while (rset.next()) {
> i++;
> System.out.println("Row" + i + " " + rset.getString(2));
> }
>
> Thomas
>
>
>
>
> -------------------------------------------------------
> SF email is sponsored by - The IT Product Guide
> Read honest & candid reviews on hundreds of IT Products from real
> users.
> Discover which products truly live up to the hype. Start reading now.
> http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
> _______________________________________________
> Springframework-developer mailing list
> Spr...@li...
> https://lists.sourceforge.net/lists/listinfo/springframework-developer
>
>
>
> -------------------------------------------------------
> SF email is sponsored by - The IT Product Guide
> Read honest & candid reviews on hundreds of IT Products from real
> users.
> Discover which products truly live up to the hype. Start reading now.
> http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
> _______________________________________________
> Springframework-developer mailing list
> Spr...@li...
> https://lists.sourceforge.net/lists/listinfo/springframework-developer
>
>
-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
Springframework-developer mailing list
Spr...@li...
https://lists.sourceforge.net/lists/listinfo/springframework-developer
|
