Re: [infinispan-dev] RemoteCache vs BasicCache

On 9 July 2013 18:01, Mircea Markus <mmarkus(a)redhat.com&gt; wrote: > > On 9 Jul 2013, at 13:47, Sanne Grinovero <sanne(a)infinispan.org&gt; wrote: > >> Indeed that's an excellent idea. But I would add also a common parent >> interface having the basic functionality common to all. > > If ConcurrentMap doesn't have what we need, of course :-) ConcurrentMap adds a load of pitfalls which should not be there: all the methods whose result is depending on the node they are run on. I know we make it clear in javadocs and several warnings on the doc, but experience tells that this is not good enough :-( I'd propose to remove them all, and provide an adapter layer (optional) for those needing to replace a ConcurrentMap in their code. Methods like "size()" - as discussed before - are possibly useful for statistics and diagnostics, and therefore should be accessed in a different explicit way. cache.diagnosticsStatistics().localContainerSize()