Drop unnecessary Supplier wrapper when configuring web routing.

wildfly · Oct 31, 2017 · df79d29 · df79d29
1 parent 1ea5128
commit df79d29
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 8 deletions.
diff --git a/.../org/wildfly/clustering/web/infinispan/session/InfinispanRouteLocatorBuilderProvider.java b/.../org/wildfly/clustering/web/infinispan/session/InfinispanRouteLocatorBuilderProvider.java
@@ -25,7 +25,6 @@
 import java.util.LinkedList;
 import java.util.List;
 import java.util.ServiceLoader;
-import java.util.function.Supplier;
 
 import org.infinispan.configuration.cache.CacheMode;
 import org.jboss.as.clustering.controller.CapabilityServiceBuilder;
@@ -55,12 +54,12 @@ public CapabilityServiceBuilder<RouteLocator> getRouteLocatorBuilder(String serv
     }
 
     @Override
-    public Collection<CapabilityServiceBuilder<?>> getRouteLocatorConfigurationBuilders(String serverName, Supplier<ValueDependency<String>> routeDependencyProvider) {
+    public Collection<CapabilityServiceBuilder<?>> getRouteLocatorConfigurationBuilders(String serverName, ValueDependency<String> routeDependency) {
         String containerName = InfinispanSessionManagerFactoryBuilder.DEFAULT_CACHE_CONTAINER;
 
         List<CapabilityServiceBuilder<?>> builders = new LinkedList<>();
 
-        builders.add(new RouteRegistryEntryProviderBuilder(serverName, routeDependencyProvider.get()));
+        builders.add(new RouteRegistryEntryProviderBuilder(serverName, routeDependency));
         builders.add(new TemplateConfigurationBuilder(ServiceName.parse(InfinispanCacheRequirement.CONFIGURATION.resolve(containerName, serverName)), containerName, serverName, null, builder -> {
             CacheMode mode = builder.clustering().cacheMode();
             builder.clustering().cacheMode(mode.isClustered() ? CacheMode.REPL_SYNC : CacheMode.LOCAL);

diff --git a/...web/spi/src/main/java/org/wildfly/clustering/web/session/RouteLocatorBuilderProvider.java b/...web/spi/src/main/java/org/wildfly/clustering/web/session/RouteLocatorBuilderProvider.java
@@ -22,7 +22,6 @@
 package org.wildfly.clustering.web.session;
 
 import java.util.Collection;
-import java.util.function.Supplier;
 
 import org.jboss.as.clustering.controller.CapabilityServiceBuilder;
 import org.wildfly.clustering.service.ValueDependency;
@@ -44,5 +43,5 @@ public interface RouteLocatorBuilderProvider {
      * @param route the injected route source
      * @return a service builder
      */
-    Collection<CapabilityServiceBuilder<?>> getRouteLocatorConfigurationBuilders(String serverName, Supplier<ValueDependency<String>> routeDependencyProvider);
+    Collection<CapabilityServiceBuilder<?>> getRouteLocatorConfigurationBuilders(String serverName, ValueDependency<String> routeDependencyProvider);
 }
diff --git a/...y/clustering/web/undertow/session/DistributableSessionIdentifierCodecBuilderProvider.java b/...y/clustering/web/undertow/session/DistributableSessionIdentifierCodecBuilderProvider.java
@@ -26,7 +26,6 @@
 import java.util.LinkedList;
 import java.util.Optional;
 import java.util.ServiceLoader;
-import java.util.function.Supplier;
 import java.util.stream.StreamSupport;
 
 import org.jboss.as.clustering.controller.CapabilityServiceBuilder;
@@ -57,8 +56,8 @@ public Collection<CapabilityServiceBuilder<?>> getServerBuilders(String serverNa
         Collection<CapabilityServiceBuilder<?>> builders = new LinkedList<>();
         CapabilityServiceBuilder<String> routeBuilder = new RouteBuilder(serverName);
         builders.add(routeBuilder);
-        Supplier<ValueDependency<String>> routeDependencyProvider = () -> new InjectedValueDependency<>(routeBuilder, String.class);
-        PROVIDER.ifPresent(provider -> builders.addAll(provider.getRouteLocatorConfigurationBuilders(serverName, routeDependencyProvider)));
+        ValueDependency<String> routeDependency = new InjectedValueDependency<>(routeBuilder, String.class);
+        PROVIDER.ifPresent(provider -> builders.addAll(provider.getRouteLocatorConfigurationBuilders(serverName, routeDependency)));
         return builders;
     }
 }