Re: [infinispan-dev] About size()

2014-10-06 Thread Sanne Grinovero
On 3 October 2014 18:38, Dennis Reed der...@redhat.com wrote: Since size() is defined by the ConcurrentMap interface, it already has a precisely defined meaning. The only correct implementation is E. +1 The current non-correct implementation was just because it's expensive to calculate

Re: [infinispan-dev] About size()

2014-10-06 Thread Tristan Tarrant
I think we should provide correct implementations of size() (and others) and provide shortcut implementations using our usual Flag API (e.g. SKIP_REMOTE_LOOKUP). Tristan On 06/10/14 12:57, Sanne Grinovero wrote: On 3 October 2014 18:38, Dennis Reed der...@redhat.com wrote: Since size() is

Re: [infinispan-dev] About size()

2014-10-06 Thread Sanne Grinovero
On 6 October 2014 12:44, Tristan Tarrant ttarr...@redhat.com wrote: I think we should provide correct implementations of size() (and others) and provide shortcut implementations using our usual Flag API (e.g. SKIP_REMOTE_LOOKUP). Right that would be very nice. Same for CacheStore interaction: