From fd54b4e4b136b05b67b471dca3ca6ce3dd116710 Mon Sep 17 00:00:00 2001 From: Henning Baldersheim Date: Sun, 4 Dec 2022 22:42:13 +0100 Subject: Revert collect(Collectors.toList()) --- .../main/java/com/yahoo/vespa/service/duper/InfraApplication.java | 2 +- .../java/com/yahoo/vespa/service/duper/DuperModelManagerTest.java | 6 +++--- .../src/test/java/com/yahoo/vespa/service/duper/DuperModelTest.java | 2 +- .../com/yahoo/vespa/service/health/HealthMonitorManagerTest.java | 2 +- .../java/com/yahoo/vespa/service/health/StateV1HealthModelTest.java | 2 +- .../yahoo/vespa/service/model/ApplicationInstanceGeneratorTest.java | 2 +- .../src/test/java/com/yahoo/vespa/service/model/ExampleModel.java | 4 ++-- .../test/java/com/yahoo/vespa/service/model/ExampleModelTest.java | 4 ++-- .../yahoo/vespa/service/model/ServiceHostListenerAdapterTest.java | 2 +- .../java/com/yahoo/vespa/service/model/ServiceMonitorImplTest.java | 2 +- .../test/java/com/yahoo/vespa/service/monitor/ConfigserverUtil.java | 2 +- 11 files changed, 15 insertions(+), 15 deletions(-) (limited to 'service-monitor') diff --git a/service-monitor/src/main/java/com/yahoo/vespa/service/duper/InfraApplication.java b/service-monitor/src/main/java/com/yahoo/vespa/service/duper/InfraApplication.java index e82e2cc0265..eccffa27930 100644 --- a/service-monitor/src/main/java/com/yahoo/vespa/service/duper/InfraApplication.java +++ b/service-monitor/src/main/java/com/yahoo/vespa/service/duper/InfraApplication.java @@ -94,7 +94,7 @@ public abstract class InfraApplication implements InfraApplicationApi { } public ApplicationInfo makeApplicationInfo(List hostnames) { - List hostInfos = hostnames.stream().map(this::makeHostInfo).toList(); + List hostInfos = hostnames.stream().map(this::makeHostInfo).collect(Collectors.toList()); return new ApplicationInfo(application.id(), 0, new HostsModel(hostInfos)); } diff --git a/service-monitor/src/test/java/com/yahoo/vespa/service/duper/DuperModelManagerTest.java b/service-monitor/src/test/java/com/yahoo/vespa/service/duper/DuperModelManagerTest.java index 6cf7d4d9f26..d02cad1c068 100644 --- a/service-monitor/src/test/java/com/yahoo/vespa/service/duper/DuperModelManagerTest.java +++ b/service-monitor/src/test/java/com/yahoo/vespa/service/duper/DuperModelManagerTest.java @@ -64,7 +64,7 @@ public class DuperModelManagerTest { makeManager(false); ApplicationId id = proxyHostApplication.getApplicationId(); - List proxyHostHosts = Stream.of("proxyhost1", "proxyhost2").map(HostName::of).toList(); + List proxyHostHosts = Stream.of("proxyhost1", "proxyhost2").map(HostName::of).collect(Collectors.toList()); verify(duperModel, times(0)).add(any()); manager.infraApplicationActivated(id, proxyHostHosts); verify(duperModel, times(1)).add(any()); @@ -91,12 +91,12 @@ public class DuperModelManagerTest { } private void testEnabledConfigServerLikeInfraApplication(ApplicationId firstId, ApplicationId secondId) { - List hostnames1 = Stream.of("node11", "node12").map(HostName::of).toList(); + List hostnames1 = Stream.of("node11", "node12").map(HostName::of).collect(Collectors.toList()); manager.infraApplicationActivated(firstId, hostnames1); verify(duperModel, times(1)).add(any()); // Adding the second config server like application will be ignored - List hostnames2 = Stream.of("node21", "node22").map(HostName::of).toList(); + List hostnames2 = Stream.of("node21", "node22").map(HostName::of).collect(Collectors.toList()); assertThrows(IllegalArgumentException.class, () -> manager.infraApplicationActivated(secondId, hostnames2)); verify(duperModel, times(1)).add(any()); diff --git a/service-monitor/src/test/java/com/yahoo/vespa/service/duper/DuperModelTest.java b/service-monitor/src/test/java/com/yahoo/vespa/service/duper/DuperModelTest.java index 73a49ca8717..69d3ef4d498 100644 --- a/service-monitor/src/test/java/com/yahoo/vespa/service/duper/DuperModelTest.java +++ b/service-monitor/src/test/java/com/yahoo/vespa/service/duper/DuperModelTest.java @@ -57,7 +57,7 @@ public class DuperModelTest { List hostInfos = Arrays.stream(hostnames) .map(hostname -> new HostInfo(hostname.value(), List.of())) - .toList(); + .collect(Collectors.toList()); when(model.getHosts()).thenReturn(hostInfos); } diff --git a/service-monitor/src/test/java/com/yahoo/vespa/service/health/HealthMonitorManagerTest.java b/service-monitor/src/test/java/com/yahoo/vespa/service/health/HealthMonitorManagerTest.java index f7ee64b52a6..c77ff519abd 100644 --- a/service-monitor/src/test/java/com/yahoo/vespa/service/health/HealthMonitorManagerTest.java +++ b/service-monitor/src/test/java/com/yahoo/vespa/service/health/HealthMonitorManagerTest.java @@ -48,7 +48,7 @@ public class HealthMonitorManagerTest { public void infrastructureApplication() { ProxyHostApplication proxyHostApplication = new ProxyHostApplication(); when(duperModel.isSupportedInfraApplication(proxyHostApplication.getApplicationId())).thenReturn(true); - List hostnames = Stream.of("proxyhost1", "proxyhost2").map(HostName::of).toList(); + List hostnames = Stream.of("proxyhost1", "proxyhost2").map(HostName::of).collect(Collectors.toList()); ApplicationInfo proxyHostApplicationInfo = proxyHostApplication.makeApplicationInfo(hostnames); manager.applicationActivated(proxyHostApplicationInfo); diff --git a/service-monitor/src/test/java/com/yahoo/vespa/service/health/StateV1HealthModelTest.java b/service-monitor/src/test/java/com/yahoo/vespa/service/health/StateV1HealthModelTest.java index 18b56f76e88..8ae0e55fe41 100644 --- a/service-monitor/src/test/java/com/yahoo/vespa/service/health/StateV1HealthModelTest.java +++ b/service-monitor/src/test/java/com/yahoo/vespa/service/health/StateV1HealthModelTest.java @@ -38,7 +38,7 @@ public class StateV1HealthModelTest { private final Duration requestTimeout = Duration.ofSeconds(2); private final Duration keepAlive = Duration.ofSeconds(3); private final ProxyHostApplication proxyHostApplication = new ProxyHostApplication(); - private final List hostnames = Stream.of("host1", "host2").map(HostName::of).toList(); + private final List hostnames = Stream.of("host1", "host2").map(HostName::of).collect(Collectors.toList()); private final ApplicationInfo proxyHostApplicationInfo = proxyHostApplication.makeApplicationInfo(hostnames); private final StateV1HealthModel model = new StateV1HealthModel(healthStaleness, requestTimeout, keepAlive, executor); diff --git a/service-monitor/src/test/java/com/yahoo/vespa/service/model/ApplicationInstanceGeneratorTest.java b/service-monitor/src/test/java/com/yahoo/vespa/service/model/ApplicationInstanceGeneratorTest.java index a2532590f52..8f87a7bad68 100644 --- a/service-monitor/src/test/java/com/yahoo/vespa/service/model/ApplicationInstanceGeneratorTest.java +++ b/service-monitor/src/test/java/com/yahoo/vespa/service/model/ApplicationInstanceGeneratorTest.java @@ -34,7 +34,7 @@ public class ApplicationInstanceGeneratorTest { when(statusProvider.getStatus(any(), any(), any(), any())).thenReturn(new ServiceStatusInfo(ServiceStatus.NOT_CHECKED)); Zone zone = mock(Zone.class); ApplicationInfo configServer = configServerApplication.makeApplicationInfo( - configServerList.stream().map(HostName::of).toList()); + configServerList.stream().map(HostName::of).collect(Collectors.toList())); ApplicationInstance applicationInstance = new ApplicationInstanceGenerator(configServer, zone) .makeApplicationInstance(statusProvider); diff --git a/service-monitor/src/test/java/com/yahoo/vespa/service/model/ExampleModel.java b/service-monitor/src/test/java/com/yahoo/vespa/service/model/ExampleModel.java index 8e1f2349046..508c98d1292 100644 --- a/service-monitor/src/test/java/com/yahoo/vespa/service/model/ExampleModel.java +++ b/service-monitor/src/test/java/com/yahoo/vespa/service/model/ExampleModel.java @@ -35,7 +35,7 @@ public class ExampleModel { public static final ApplicationId APPLICATION_ID = ApplicationId.from(TENANT, APPLICATION_NAME, INSTANCE_NAME); public static SuperModel createExampleSuperModelWithOneRpcPort(String hostname, int rpcPort) { - List hosts = Stream.of(hostname).toList(); + List hosts = Stream.of(hostname).collect(Collectors.toList()); ApplicationInfo applicationInfo = ExampleModel .createApplication(TENANT, APPLICATION_NAME) @@ -83,7 +83,7 @@ public class ExampleModel { List allHosts = clusters.stream() .flatMap(clusterBuilder -> clusterBuilder.hosts.stream()) .distinct() - .toList(); + .collect(Collectors.toList()); List hostInfos = new ArrayList<>(); for (String hostname : allHosts) { diff --git a/service-monitor/src/test/java/com/yahoo/vespa/service/model/ExampleModelTest.java b/service-monitor/src/test/java/com/yahoo/vespa/service/model/ExampleModelTest.java index 829b1fe5859..b66c6968764 100644 --- a/service-monitor/src/test/java/com/yahoo/vespa/service/model/ExampleModelTest.java +++ b/service-monitor/src/test/java/com/yahoo/vespa/service/model/ExampleModelTest.java @@ -30,8 +30,8 @@ public class ExampleModelTest { @Test public void test() { - List contentNodes = Stream.of("host1", "host2").toList(); - List containerNodes = Stream.of("host3", "host4").toList(); + List contentNodes = Stream.of("host1", "host2").collect(Collectors.toList()); + List containerNodes = Stream.of("host3", "host4").collect(Collectors.toList()); ApplicationInfo application = ExampleModel .createApplication( diff --git a/service-monitor/src/test/java/com/yahoo/vespa/service/model/ServiceHostListenerAdapterTest.java b/service-monitor/src/test/java/com/yahoo/vespa/service/model/ServiceHostListenerAdapterTest.java index c93e4dfb7fe..97b6ff0c3e2 100644 --- a/service-monitor/src/test/java/com/yahoo/vespa/service/model/ServiceHostListenerAdapterTest.java +++ b/service-monitor/src/test/java/com/yahoo/vespa/service/model/ServiceHostListenerAdapterTest.java @@ -142,7 +142,7 @@ public class ServiceHostListenerAdapterTest { List hostnameList = Stream.of(hostnames) .map(hostname -> new HostInfo(hostname, List.of())) - .toList(); + .collect(Collectors.toList()); when(model.getHosts()).thenReturn(hostnameList); return applicationInfo; diff --git a/service-monitor/src/test/java/com/yahoo/vespa/service/model/ServiceMonitorImplTest.java b/service-monitor/src/test/java/com/yahoo/vespa/service/model/ServiceMonitorImplTest.java index c3f805b0572..c98e9ae2b18 100644 --- a/service-monitor/src/test/java/com/yahoo/vespa/service/model/ServiceMonitorImplTest.java +++ b/service-monitor/src/test/java/com/yahoo/vespa/service/model/ServiceMonitorImplTest.java @@ -31,7 +31,7 @@ public class ServiceMonitorImplTest { ApplicationInfo application1 = mock(ApplicationInfo.class); ApplicationInfo application2 = mock(ApplicationInfo.class); List applications = Stream.of(application1, application2) - .toList(); + .collect(Collectors.toList()); when(duperModelManager.getApplicationInfos()).thenReturn(applications); ServiceModel serviceModel = serviceMonitor.getServiceModelSnapshot(); diff --git a/service-monitor/src/test/java/com/yahoo/vespa/service/monitor/ConfigserverUtil.java b/service-monitor/src/test/java/com/yahoo/vespa/service/monitor/ConfigserverUtil.java index 498a9dfc15e..df489a8c336 100644 --- a/service-monitor/src/test/java/com/yahoo/vespa/service/monitor/ConfigserverUtil.java +++ b/service-monitor/src/test/java/com/yahoo/vespa/service/monitor/ConfigserverUtil.java @@ -37,7 +37,7 @@ public class ConfigserverUtil { String configServerHostname3) { return new ConfigServerApplication().makeApplicationInfo( Stream.of(configServerHostname1, configServerHostname2, configServerHostname3) - .map(HostName::of).toList()); + .map(HostName::of).collect(Collectors.toList())); } public static ApplicationInfo makeExampleConfigServer() { -- cgit v1.2.3