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
[email protected]
twitter.com/galderz

Project Lead, Escalante
http://escalante.io

Engineer, Infinispan
http://infinispan.org


_______________________________________________
infinispan-dev mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/infinispan-dev

Reply via email to