From 67926d3509aaff8bb7a03b732c29a9fe5c7d03fd Mon Sep 17 00:00:00 2001 From: Jon Bratseth Date: Thu, 24 Jan 2019 16:19:53 +0100 Subject: Nonfunctional changes only --- .../yahoo/vespa/config/server/model/LbServicesProducer.java | 10 ++++------ .../com/yahoo/vespa/config/server/model/RoutingProducer.java | 1 + .../vespa/config/server/model/LbServicesProducerTest.java | 2 +- 3 files changed, 6 insertions(+), 7 deletions(-) (limited to 'configserver') diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/model/LbServicesProducer.java b/configserver/src/main/java/com/yahoo/vespa/config/server/model/LbServicesProducer.java index 20e8c664761..7835ab2cf03 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/model/LbServicesProducer.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/model/LbServicesProducer.java @@ -60,18 +60,16 @@ public class LbServicesProducer implements LbServicesConfig.Producer { ab.activeRotation(getActiveRotation(app)); app.getModel().getHosts().stream() .sorted((a, b) -> a.getHostname().compareTo(b.getHostname())) - .forEach(hostInfo -> { - ab.hosts(hostInfo.getHostname(), getHostsConfig(hostInfo)); - }); + .forEach(hostInfo -> ab.hosts(hostInfo.getHostname(), getHostsConfig(hostInfo))); return ab; } private boolean getActiveRotation(ApplicationInfo app) { boolean activeRotation = false; for (HostInfo hostInfo : app.getModel().getHosts()) { - final Optional container = hostInfo.getServices().stream().filter( + Optional container = hostInfo.getServices().stream().filter( serviceInfo -> serviceInfo.getServiceType().equals("container") || - serviceInfo.getServiceType().equals("qrserver")). + serviceInfo.getServiceType().equals("qrserver")). findAny(); if (container.isPresent()) { activeRotation |= Boolean.valueOf(container.get().getProperty("activeRotation").orElse("false")); @@ -91,7 +89,7 @@ public class LbServicesProducer implements LbServicesConfig.Producer { } private LbServicesConfig.Tenants.Applications.Hosts.Services.Builder getServiceConfig(ServiceInfo serviceInfo) { - final List endpointAliases = Stream.of(serviceInfo.getProperty("endpointaliases").orElse("").split(",")). + List endpointAliases = Stream.of(serviceInfo.getProperty("endpointaliases").orElse("").split(",")). filter(prop -> !"".equals(prop)).collect(Collectors.toList()); endpointAliases.addAll(Stream.of(serviceInfo.getProperty("rotations").orElse("").split(",")).filter(prop -> !"".equals(prop)).collect(Collectors.toList())); Collections.sort(endpointAliases); diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/model/RoutingProducer.java b/configserver/src/main/java/com/yahoo/vespa/config/server/model/RoutingProducer.java index 236cb4e9b97..7c85b51c920 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/model/RoutingProducer.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/model/RoutingProducer.java @@ -18,6 +18,7 @@ import java.util.Set; * @author Christian Andersen */ public class RoutingProducer implements RoutingConfig.Producer { + static final ApplicationName ROUTING_APPLICATION = ApplicationName.from("routing"); private final Map> models; diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/model/LbServicesProducerTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/model/LbServicesProducerTest.java index 35b6e4cf644..e7f0d842999 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/model/LbServicesProducerTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/model/LbServicesProducerTest.java @@ -88,7 +88,7 @@ public class LbServicesProducerTest { } private LbServicesConfig createModelAndGetLbServicesConfig(RegionName regionName) throws IOException, SAXException { - final Zone zone = new Zone(Environment.prod, regionName); + Zone zone = new Zone(Environment.prod, regionName); Map> testModel = createTestModel(new DeployState.Builder() .zone(zone) .properties(new DeployProperties.Builder().build()) -- cgit v1.2.3