On Thu, Feb 24, 2011 at 2:01 PM, David Winslow <[email protected]> wrote:
> On Wed, Feb 23, 2011 at 8:52 PM, Justin Deoliveira > <[email protected]>wrote: > >> I think the logic looks ok in in LocalWorkspaceResourceAccessManager based >> on my limited understanding of the new api. Unless I am mistaken however >> the intersection(WorkspaceAccessLimits,WorkspaceAccessLimits) method is >> unused? >> > > That's true (it's not used) but I moved these to the > ResourceAccessManagerWrapper and left it in for consistency. I also made > them and the LocalWorkspaceResourceAccessManager aware of the narrower types > of DataAccessLimits. An updated patch is attached to the same ticket. > > One concern I had was in finding the intersection of the ROI for > CoverageAccessInfo - the intersection of two MultiPolygons may be a Polygon > (or other types? I am not sure). I suspect somewhere in GeoTools there is a > method to correct for this, but in my patch I simply put in an if/else > statement and ignored the possibility that they might intersect in a point > or something. > > A very unlucky intersection may end up being a mixed collection with polygons, lines and points. So yeah, some code should be used to clean that up and get back to a multipolygon 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 mob: +39 333 8128928 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 -------------------------------------------------------
------------------------------------------------------------------------------ Free Software Download: Index, Search & Analyze Logs and other IT data in Real-Time with Splunk. Collect, index and harness all the fast moving IT data generated by your applications, servers and devices whether physical, virtual or in the cloud. Deliver compliance at lower cost and gain new business insights. http://p.sf.net/sfu/splunk-dev2dev
_______________________________________________ Geoserver-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geoserver-devel
