Gabriel Roldan ha scritto: > Justin, all > > during my last weeks of arcsde gce stress testing, I found a potential > concurrency penalty in ResourcePool.getGridCoverageReader. If hit with > various concurrent requests at it may instantiate lots of readers to > ultimately leave a single one cached. > The following patch applies the same trick than for > ResourcePool.getDataStore, please let me know if it seems appropriate to > you: <http://pastebin.com/m32ebb87b> > > It just double checks for the cached reader after acquiring the lock on > the cache.
The patch looks good to me. Maybe remove those capital letters comments ;-) (I noticed they are also in the original code, not sure why...) Cheers Andrea -- Andrea Aime OpenGeo - http://opengeo.org Expert service straight from the developers. ------------------------------------------------------------------------------ Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day trial. Simplify your report design, integration and deployment - and focus on what you do best, core application coding. Discover what's new with Crystal Reports now. http://p.sf.net/sfu/bobj-july _______________________________________________ Geoserver-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geoserver-devel
