Re: [infinispan-dev] cache loader: purgerThreads and purgeSynchronously

On 14 August 2013 15:39, Manik Surtani <msurtani(a)redhat.com&gt; wrote: > I agree with you, Sanne. +1 always good :P > But I think from a config perspective, this does need an overhaul. +1 > I think the correct approach - given some of the newly forming ideas on the CacheLoader/CacheStore SPI we've been having - is to not specify whether purging is synchronous or not +1 that is implementation specific, some might even not need a purger at all as - for example - a remote Infinispan store does understand how to cleanup by itself. > and not to specify a number of threads, but instead to point to a named executor. > E.g., the way we define executors for async notification, async transport, etc. +1 sounds good and consistent, but let's improve even more on consistency: If we're reorganizing the CacheLoader configuration schema,

I would very much like it to finally have named CacheStore/Loader elements like we do with Executors, and being able to refer to them from Caches rather than nesting their configuration in a Cache element. If you ever configured - for example - a JDBCCacheStore for at least 3 caches in the same CacheManager, you know how much this is desirable. Mircea: we've spoken about that option several times, AFAIR your main point to not do that (yet) was to not introduce changes in the configuration schema.

Since it seems you're restructuring it all, let's do it all :-)