-1 to removing them ...
On 26 Apr 2013, at 14:58, Galder Zamarreño <galder(a)redhat.com> wrote:
Hi all,
Looking at the pull req for ISPN-2281, I'm adding methods like this to
AdvancedCache:
V put(K key, V value, Metadata metadata);
Now, I think that adding this should deprecate these methods:
V put(K key, V value, long lifespan, TimeUnit unit);
V put(K key, V value, long lifespan, TimeUnit lifespanUnit, long maxIdleTime, TimeUnit
maxIdleTimeUnit);
But these two are located in BasicCache, and if we end up removing them eventually,
it'd mean that either:
a) users need to use advanced cache even for adding lifespan
b) put(k, v, metadata) gets promoted to BasicCache
I don't like either option tbh. BasicCache might be implemented by users or AS guys,
which until we have JDK8 (in year 2048 :p), will break existing clients, and not happy for
people to get hold of AdvancedCache for use cases where they didn't need before.
Cheers,
--
Galder Zamarreño
galder(a)redhat.com
twitter.com/galderz
Project Lead, Escalante
http://escalante.io
Engineer, Infinispan
http://infinispan.org
_______________________________________________
infinispan-dev mailing list
infinispan-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/infinispan-dev