On 10 Feb 2015 at 10:07:22, Marius Dumitru Florea ([email protected](mailto:[email protected])) wrote: [snip]
Option 1: =========
GroupManager |_ Iterator getAllUsers()
(in the future we would also have a getMatchingUsers(…))
By default the impl would retrieve batches of user references from the group (say 100 at a time)
However, if you want to get just 1 user for example, this is a bit overkill to get 100 DB results. But is that a real use case? So we also have option 2:
Option 2: ==========
GroupManager |_ GroupResultSet getAllUsers()
Where:
GroupResultSet |_ Iterator iterator() - default to batch size of 100 |_ Iterator iterator(int batchSize)
|_ Collection get(int nb, int offset) (do we really need this api?)
Definitely. For an API that may return a very large list we need to be able to paginate the display (e.g. group members live table).
Indeed, good point. Thanks -Vincent [snip]