On 26 Jun 2008, at 13:48, Emmanuel Bernard wrote: <SNIP /> Then why don't we just lazily load Lucene data into the iterator and drop scroll()? I see supporting both as superfluous. And iterator() is more familiar to devs. +1, good point. We'd need a close() method on the QueryResultIterator. I see just returning a null if the object is not in the cache as something people would expect. If you think eager loading of Lucene data is something people would find useful, we could have 2 separate implementations of the QueryResultIterator - a QRILazyImpl and a QRIEagerImpl. And the interface to obtain these could be on the CacheQuery interface: CacheQuery.iterator(boolean lazy); Yup. :-) -- Manik Surtani Lead, JBoss Cache manik(a)jboss.org