From 8c78a5e9a6d3ba33454a7a47b015b5134ffafa18 Mon Sep 17 00:00:00 2001
From: Bojan Serafimov <bojan@pop-os.localdomain>
Date: Wed, 25 May 2022 11:31:22 -0400
Subject: [PATCH] Clippy

---
 pageserver/src/tenant_jobs/worker.rs | 9 +++++++++
 pageserver/src/thread_mgr.rs         | 2 +-
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/pageserver/src/tenant_jobs/worker.rs b/pageserver/src/tenant_jobs/worker.rs
index d8ad72b377..d093137c26 100644
--- a/pageserver/src/tenant_jobs/worker.rs
+++ b/pageserver/src/tenant_jobs/worker.rs
@@ -130,6 +130,15 @@ where
     condvar: Arc<Condvar>, // Notified when idle worker should wake up
 }
 
+impl<J: Job> Default for Pool<J>
+where
+    J::ErrorType: Debug,
+{
+    fn default() -> Self {
+        Self::new()
+    }
+}
+
 impl<J: Job> Pool<J>
 where
     J::ErrorType: Debug,
diff --git a/pageserver/src/thread_mgr.rs b/pageserver/src/thread_mgr.rs
index a30eae2be4..1d0ea01a52 100644
--- a/pageserver/src/thread_mgr.rs
+++ b/pageserver/src/thread_mgr.rs
@@ -60,7 +60,7 @@ lazy_static! {
     /// Global registry of threads
     static ref THREADS: Mutex<HashMap<u64, Arc<PageServerThread>>> = Mutex::new(HashMap::new());
 
-    // TODO make these per thread?
+    // TODO make these per thread for targetted shutdown, also for cleanup.
     /// Condvars to notify after shutdown request
     static ref SHUTDOWN_CONDVARS: Mutex<Vec<Arc<Condvar>>> = Mutex::new(Vec::new());
 }