From dc90003fd77995596c226c23710f4d0126ecc5e5 Mon Sep 17 00:00:00 2001 From: Paul Ferraro Date: Sat, 31 Mar 2018 13:55:29 -0400 Subject: [PATCH] Remove redundant allocation of ConfigurationBuilder. --- .../subsystem/ClusteredCacheBuilder.java | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/clustering/infinispan/extension/src/main/java/org/jboss/as/clustering/infinispan/subsystem/ClusteredCacheBuilder.java b/clustering/infinispan/extension/src/main/java/org/jboss/as/clustering/infinispan/subsystem/ClusteredCacheBuilder.java index 0fabd63049a7..983bb4308c40 100644 --- a/clustering/infinispan/extension/src/main/java/org/jboss/as/clustering/infinispan/subsystem/ClusteredCacheBuilder.java +++ b/clustering/infinispan/extension/src/main/java/org/jboss/as/clustering/infinispan/subsystem/ClusteredCacheBuilder.java @@ -24,9 +24,9 @@ import static org.jboss.as.clustering.infinispan.subsystem.ClusteredCacheResourceDefinition.Attribute.*; +import java.util.concurrent.TimeUnit; + import org.infinispan.configuration.cache.CacheMode; -import org.infinispan.configuration.cache.ClusteringConfiguration; -import org.infinispan.configuration.cache.ClusteringConfigurationBuilder; import org.infinispan.configuration.cache.Configuration; import org.infinispan.configuration.cache.ConfigurationBuilder; import org.jboss.as.controller.OperationContext; @@ -43,7 +43,7 @@ public class ClusteredCacheBuilder extends CacheConfigurationBuilder { private final CacheMode mode; - private volatile ClusteringConfiguration clustering; + private volatile long remoteTimeout; ClusteredCacheBuilder(PathAddress address, CacheMode mode) { super(address); @@ -52,18 +52,16 @@ public class ClusteredCacheBuilder extends CacheConfigurationBuilder { @Override public Builder configure(OperationContext context, ModelNode model) throws OperationFailedException { - ClusteringConfigurationBuilder builder = new ConfigurationBuilder().clustering().cacheMode(this.mode); - - builder.remoteTimeout(REMOTE_TIMEOUT.resolveModelAttribute(context, model).asLong()); - - this.clustering = builder.create(); - + this.remoteTimeout = REMOTE_TIMEOUT.resolveModelAttribute(context, model).asLong(); return super.configure(context, model); } @Override public void accept(ConfigurationBuilder builder) { - builder.clustering().read(this.clustering); + builder.clustering() + .cacheMode(this.mode) + .remoteTimeout(this.remoteTimeout, TimeUnit.MILLISECONDS) + ; super.accept(builder); } }