From 84233b1717c4ee772b7dce6f5a22d40a47c7d9d0 Mon Sep 17 00:00:00 2001 From: Martin Polden Date: Fri, 21 Sep 2018 17:29:45 +0200 Subject: Revert to single-threaded update of metrics The implementation of `MetricsService` doesn't seem to handle multiple threads well, need to look into it. Revert to single thread for now. --- .../hosted/controller/maintenance/DeploymentMetricsMaintainer.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'controller-server/src') diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentMetricsMaintainer.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentMetricsMaintainer.java index 6ea9d548367..03123d281a5 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentMetricsMaintainer.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentMetricsMaintainer.java @@ -51,7 +51,7 @@ public class DeploymentMetricsMaintainer extends Maintainer { // Run parallel stream inside a custom ForkJoinPool so that we can control the number of threads used ForkJoinPool pool = new ForkJoinPool(applicationsToUpdateInParallel); pool.submit(() -> { - applicationList.parallelStream().forEach(application -> { + applicationList.stream().forEach(application -> { try { applications.lockIfPresent(application.id(), locked -> applications.store(locked.with(controller().metricsService().getApplicationMetrics(application.id())))); -- cgit v1.2.3