From 626fe95d89b0afd86b03b6ad6df137d501f2ff8a Mon Sep 17 00:00:00 2001
From: James Wilson <james@jsdw.me>
Date: Mon, 9 Aug 2021 10:13:10 +0100
Subject: [PATCH] 1 aggregator loop by default for now

---
 backend/telemetry_core/src/main.rs | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/backend/telemetry_core/src/main.rs b/backend/telemetry_core/src/main.rs
index aeee8f6..4fe7278 100644
--- a/backend/telemetry_core/src/main.rs
+++ b/backend/telemetry_core/src/main.rs
@@ -90,9 +90,9 @@ fn main() {
     let num_aggregators = match opts.num_aggregators {
         Some(0) => num_cpus::get(),
         Some(n) => n,
-        // By default, we'll have half as many aggregator tasks
-        // running as we do worker threads (minimum 1).
-        None => usize::max(worker_threads / 2, 1),
+        // For now, we just have 1 aggregator loop by default,
+        // but we may want to be smarter here eventually.
+        None => 1,
     };
 
     tokio::runtime::Builder::new_multi_thread()