On Sun, Nov 4, 2012 at 8:24 AM, Jody Garnett <[email protected]> wrote: > I had a go and ProcessingCollection extends BaseFeatureCollection, and > arrived at one critical implementation difference. > ProcessingFeatureCollection takes care to lazily create the schema; I can > image for some process this work can be > fairly significant. > > Additional helper methods: > - DataUtilities.bounds( FeatureCollection ) <-- already existed > - DataUtilities.bounds( FeatureIterator ) > - DataUtilities.count( FeatureCollection ) > - DataUtilities.count( FeatureIterator ) > > Once again I hope this reduce the amount of duplication in our codebase, I > added the bounds( FeatureIterator ) method to make it clear that the > calculation was limited to traversing the features manually.
Sure, just as long as we give the convenience to the implementor, but we don't make that the default behavior: manual counting and manual bounding in a processing collection means running the actual process, I want the writer to think if that is the only possible way, since running a process may take a really long time. Btw, ProcessingCollection has its own WrappingIterator which I guess could be now replaced with BridgeIterator, if I got the right idea about what BridgeIterator is. Cheers Andrea -- == Our support, Your Success! Visit http://opensdi.geo-solutions.it for more information. == Ing. Andrea Aime @geowolf Technical Lead GeoSolutions S.A.S. Via Poggio alle Viti 1187 55054 Massarosa (LU) Italy phone: +39 0584 962313 fax: +39 0584 1660272 mob: +39 339 8844549 http://www.geo-solutions.it http://twitter.com/geosolutions_it ------------------------------------------------------- ------------------------------------------------------------------------------ LogMeIn Central: Instant, anywhere, Remote PC access and management. Stay in control, update software, and manage PCs from one command center Diagnose problems and improve visibility into emerging IT issues Automate, monitor and manage. Do more in less time with Central http://p.sf.net/sfu/logmein12331_d2d _______________________________________________ GeoTools-Devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
