From: Andrea A. <and...@ge...> - 2011-08-29 15:58:35
|
On Mon, Aug 29, 2011 at 4:43 PM, Justin Deoliveira <jde...@op...> wrote: > Wow indeed, very impressive. Definitely +1 for merging it into main, > most particularly ContentDataStore, since the is the recommended starting > point for new datastores these days. Since it is a wrapping feature reader > this should be trivial. Great stuff. Hey hey, everybody seems enthusiastic. Not all that shines is gold though. Yes, this thing can allow sorting and thus transparent paging, however, if you try to page over a 20 million feature and get 10 features at page 3 you'll have to first sort all of them, and then grab the page. Which is doable, but might bring the machine on his knees. I don't have a good solution for this though... maybe expose a param in the stores to allow the local data sorting, or don't even try it? The other open issue is that yes, you can configure a max number of open files, yet that has to be multiplied by the number of concurrent requests, so on a server app it's still relatively easy to break the max file open limit. Not sure how to deal with this one either, we'd need some global counter... Suggestions? Cheers Andrea -- ------------------------------------------------------- Ing. Andrea Aime GeoSolutions S.A.S. Tech lead Via Poggio alle Viti 1187 55054 Massarosa (LU) Italy phone: +39 0584 962313 fax: +39 0584 962313 http://www.geo-solutions.it http://geo-solutions.blogspot.com/ http://www.youtube.com/user/GeoSolutionsIT http://www.linkedin.com/in/andreaaime http://twitter.com/geowolf ------------------------------------------------------- |