[infinispan-dev] Cleanup of new configuration API

[Pete Muir]

On 19 Jan 2012, at 16:14, Galder Zamarreño wrote:

> Hi all,
> 
> Re: https://issues.jboss.org/browse/ISPN-1745
> 
> The following deprecated methods belonging to the new configuration are going away for final and the methods that should be used instead:
> 
> - SerializationConfigurationBuilder.marshallerClass() -> SerializationConfigurationBuilder.marshaller()
> - ClusteringConfigurationBuilder.stateRetrieval() -> ClusteringConfigurationBuilder.stateTransfer()
> - Configuration.onePhaseCommit() -> ClusteringConfigurationBuilder.cacheMode().isSynchronous()
> - StateRetrievalConfiguration -> StateTransferConfiguration
> - StateRetrievalConfigurationBuilder -> StateTransferConfigurationBuilder
> - TransactionConfigurationBuilder.cacheStopTimeout(int) -> TransactionConfigurationBuilder.cacheStopTimeout(long)
> 
> Other stuff I've spotted:
> - TransactionConfiguration should not contain setters, those are already in TransactionConfigurationBuilder

Yes it should. These properties are runtime mutable (what was previously @Dynamic). Anything that is runtime mutable/dynamic is adjusted at runtime through a mutator on the configuration ofect.

> - TransactionConfiguration.transactionalCache() has been removed
> - Configuration.stateTransferEnabled was removed (what the heck was it doing there?)

As it refers to two configuration areas (state transfer and loaders) it was placed on the common element.

These two were both helpers methods, and I'm not sure whether I like them or not. If we don't want them, no problem from me.

> 
> If this is problematic for anyone, please reply.
> 
> Cheers,
> --
> Galder Zamarreño
> Sr. Software Engineer
> Infinispan, JBoss Cache
> 
> 
> _______________________________________________
> infinispan-dev mailing list
> infinispan-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/infinispan-dev