Guys, what does this mean for the transition of the test suite to the new (5.1+) configuration API? Will it also have to wait until 6.0?
I did a search today and there are still about 10 callers in production code for the deprecated Cache.getConfiguration() method... I didn't have time to count the callers in test code :)

Dan


On Tue, Aug 14, 2012 at 2:48 PM, Manik Surtani <manik@jboss.org> wrote:
I agree with Galder.  These will change in 6.0 anyway.

On 14 Aug 2012, at 10:02, Galder Zamarreņo <galder@redhat.com> wrote:

Hmmm, I'm not sure this is the right time to actually change the API, that'd be IMO when we redesign the cache loaders.

Instead, I'd add a new interface with the new method and extend the current CacheLoader. It's a bit messy but should only be temporary until we have an agreement on the cache loader interface.

IOW, I'd rather change once, in a major release, than change now and then change again in 6.0.

Thoughts?

On Aug 9, 2012, at 8:54 AM, Tristan Tarrant wrote:

Yes, I'm introducing interfaces: the whole org.infinispan.configuration
is quite lacking in that respect ;)

Tristan

On 08/08/2012 07:27 PM, Sanne Grinovero wrote:
I'd suggest to keep an interface in the API signature, much better
than an abstract class.

+1 to change the API, more changes are expected soon anyway.

Sanne


On 8 August 2012 18:51, Tristan Tarrant <ttarrant@redhat.com> wrote:
Hi guys,

as part of ISPN-1978, I'm migrating the Cache Loaders to the new
configuration builder APIs. I want to change the init() method of the
CacheLoaders which has the following signature

public void init(CacheLoaderConfig config, Cache<?, ?> cache,
StreamingMarshaller m) throws CacheLoaderException

to the following:

public void init(AbstractLoaderConfiguration config, Cache<?, ?> cache,
StreamingMarshaller m) throws CacheLoaderException

This will break all existing cache loaders/stores: I'll fix all the ones
bundled in the distribution, but do we need to provide compatibility
with third party loaders ?

Tristan

_______________________________________________
infinispan-dev mailing list
infinispan-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/infinispan-dev
_______________________________________________
infinispan-dev mailing list
infinispan-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/infinispan-dev

_______________________________________________
infinispan-dev mailing list
infinispan-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/infinispan-dev

--
Galder Zamarreņo
Sr. Software Engineer
Infinispan, JBoss Cache


_______________________________________________
infinispan-dev mailing list
infinispan-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/infinispan-dev

--
Manik Surtani

Founder and Project Lead, Infinispan

Platform Architect, JBoss Data Grid


_______________________________________________
infinispan-dev mailing list
infinispan-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/infinispan-dev