diff options
author | Martin Polden <mpolden@mpolden.no> | 2018-09-21 17:29:45 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-09-21 17:29:45 +0200 |
commit | 84233b1717c4ee772b7dce6f5a22d40a47c7d9d0 (patch) | |
tree | dbc84a7311be8f631519126292290ce6e37515a3 | |
parent | 2ed2e04e3ba16da9dba56df92e3cc3a82c0afdaf (diff) |
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.
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentMetricsMaintainer.java | 2 |
1 files changed, 1 insertions, 1 deletions
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())))); |