We are getting very close to feature freeze Mauro, if you would like to try
the new one we should do so *now* as the default for the upcoming beta. If
it survives I would be comfortable leaving it as the default for the 2.6
release.
Jody
Jody Garnett
On Tue, Jul 8, 2014 at 7:31 AM, Mauro Bartolomeoli <
mauro.bartolome...@geo-solutions.it> wrote:
> Hi everybody,
> recently one of our customers was having some issues with threads blocked
> inside the default LRUAuthenticationCacheImpl class (default implementation
> of the AuthenticationCache mechanism in Geoserver).
> Searching on the internet we found other users experimenting similar
> behaviours, so probably the default cache implementation is not very stable
> under heavy load or particular conditions.
>
> To solve the issue we experimented an alternative implementation using the
> guava Cache class.
> We found that guava Cache already implements all
> the LRUAuthenticationCacheImpl functionality (expiration on idle and live
> timeouts, eviction, and so on), so it was a very easy task.
> In the future we could also think to leverage advanced guava cache
> functionality (such as the automatic "loading" of missing entries).
>
> This new implementation is working better for our customer, so far.
>
> The only thing that we did not implement is support for entry by entry
> expiration times (I found that this is used in GeoServerCompositeFilter
> using request attributes, but I don't understand exactly the purpose, can
> anyone explain me why is that used?).
>
> My proposal is to add the new implementation to Geoserver main, since
> guava is already a dependency and this would lead to a simpler
> implementation than the existing one, and one that is based on a well known
> library.
>
> We could:
> - add the new implementation but leave the old one as the default (the
> new one can be enabled adding a bean definition in applicationContext.xml)
> - add the new implementation and set it as the default, to see if it
> behaves better than previous one for a larger audience (the old one could
> be reenabled using the same applicationContext.xml approach)
>
> What do you think? Is anyone interested in this?
>
> Thanks
> Mauro
>
> --
> ==
> GeoServer Professional Services from the experts! Visit
> http://goo.gl/NWWaa2 for more information.
> ==
>
> Dott. Mauro Bartolomeoli
> @mauro_bart
> Senior Software Engineer
>
> GeoSolutions S.A.S.
> Via Poggio alle Viti 1187
> 55054 Massarosa (LU)
> Italy
> phone: +39 0584 962313
> fax: +39 0584 1660272
>
> http://www.geo-solutions.it
> http://twitter.com/geosolutions_it
>
> -------------------------------------------------------
>
>
> ------------------------------------------------------------------------------
> Open source business process management suite built on Java and Eclipse
> Turn processes into business applications with Bonita BPM Community Edition
> Quickly connect people, data, and systems into organized workflows
> Winner of BOSSIE, CODIE, OW2 and Gartner awards
> http://p.sf.net/sfu/Bonitasoft
> _______________________________________________
> Geoserver-devel mailing list
> Geoserver-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geoserver-devel
>
>
------------------------------------------------------------------------------
Open source business process management suite built on Java and Eclipse
Turn processes into business applications with Bonita BPM Community Edition
Quickly connect people, data, and systems into organized workflows
Winner of BOSSIE, CODIE, OW2 and Gartner awards
http://p.sf.net/sfu/Bonitasoft
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
