Re: Deprecating PaginatedList interfaces and SqlMapClient method

2006-10-26 Thread Guido García Bernardo
I'm not a commiter so I don't if my opinion is going to be taken into 
account, but +1 !


Clinton Begin escribió:

Hi all,

I'd like to suggest that we deprecate the PaginatedList interface, all 
implementations of it, and the SqlMapClient methods that use it.


Why?

   1. It's currently overused where it probably shouldn't be used at all.
   2. It's not a flexible model, so you can't supply your own
  implementation of PaginatedList.
   3. Using queryForList() with skip and count parameters is better
  all around.

Any thoughts?

Cheers,
Clinton



--
Guido García Bernardo

Tfn. +34 983 54 89 08
ITDEUSTO - Valladolid



Re: Deprecating PaginatedList interfaces and SqlMapClient method

2006-10-23 Thread Larry Meadors

1

..fwiw, I feel the same about lazy lists and xml param/results. ;-)

Larry


On 10/23/06, Clinton Begin [EMAIL PROTECTED] wrote:

Hi all,

I'd like to suggest that we deprecate the PaginatedList interface, all
implementations of it, and the SqlMapClient methods that use it.

Why?


It's currently overused where it probably shouldn't be used at all.
It's not a flexible model, so you can't supply your own implementation of
PaginatedList.
Using queryForList() with skip and count parameters is better all around.Any
thoughts?

Cheers,
 Clinton



RE: Deprecating PaginatedList interfaces and SqlMapClient method

2006-10-23 Thread Niels Beekman








+1



Most users are tricked into
using PaginatedList because they think its an efficient way for paging. Use
of the skip and count parameters is more ResultSet-like
which is the main concern of iBATIS.



Maybe the PaginatedList implementations can
be moved to some backwards-compatibility library, so existing applications can
be upgraded without breaking stuff. I dont actually use PaginatedList,
but I can see the need for it.



Niels











From: Clinton Begin
[mailto:[EMAIL PROTECTED] 
Sent: maandag 23 oktober 2006
16:43
To: user-java@ibatis.apache.org;
dev@ibatis.apache.org
Subject: Deprecating PaginatedList
interfaces and SqlMapClient method





Hi all,

I'd like to suggest that we deprecate the PaginatedList interface, all
implementations of it, and the SqlMapClient methods that use it.

Why?


 It's currently overused where it probably
 shouldn't be used at all. 
 It's not a flexible model, so you can't supply
 your own implementation of PaginatedList.
 Using queryForList() with skip and count
 parameters is better all around.


Any thoughts?

Cheers,
Clinton








Re: Deprecating PaginatedList interfaces and SqlMapClient method

2006-10-23 Thread Jeff Butler
+1

Most users don't understand what this is really doing. 

Jeff Butler
On 10/23/06, Clinton Begin [EMAIL PROTECTED] wrote:
Hi all,I'd like to suggest that we deprecate the PaginatedList interface, all implementations of it, and the SqlMapClient methods that use it.
Why?

It's currently overused where it probably shouldn't be used at all. 
It's not a flexible model, so you can't supply your own implementation of PaginatedList.
Using queryForList() with skip and count parameters is better all around.Any thoughts?Cheers,Clinton


Re: Deprecating PaginatedList interfaces and SqlMapClient method

2006-10-23 Thread Nathan Maves
+1On 10/23/06, Jeff Butler [EMAIL PROTECTED] wrote:
+1

Most users don't understand what this is really doing. 

Jeff Butler
On 10/23/06, Clinton Begin 
[EMAIL PROTECTED] wrote:
Hi all,I'd like to suggest that we deprecate the PaginatedList interface, all implementations of it, and the SqlMapClient methods that use it.
Why?

It's currently overused where it probably shouldn't be used at all. 
It's not a flexible model, so you can't supply your own implementation of PaginatedList.
Using queryForList() with skip and count parameters is better all around.Any thoughts?Cheers,Clinton




Re: Deprecating PaginatedList interfaces and SqlMapClient method

2006-10-23 Thread Brandon Goodin
+1 On 10/23/06, Clinton Begin [EMAIL PROTECTED] wrote:
Hi all,I'd like to suggest that we deprecate the PaginatedList interface, all implementations of it, and the SqlMapClient methods that use it.Why?It's currently overused where it probably shouldn't be used at all.
It's not a flexible model, so you can't supply your own implementation of PaginatedList.Using queryForList() with skip and count parameters is better all around.Any thoughts?Cheers,

Clinton