From 244185e6e633bf7a694089c99b2077e24f3bdb0a Mon Sep 17 00:00:00 2001 From: Joonas Koivunen Date: Tue, 28 Mar 2023 10:31:54 +0300 Subject: [PATCH] doc: comment changes Co-authored-by: Christian Schwarz --- pageserver/src/bin/pageserver.rs | 3 ++- pageserver/src/disk_usage_eviction_task.rs | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/pageserver/src/bin/pageserver.rs b/pageserver/src/bin/pageserver.rs index 1400ef0e31..7dcf6465e2 100644 --- a/pageserver/src/bin/pageserver.rs +++ b/pageserver/src/bin/pageserver.rs @@ -322,7 +322,8 @@ fn start_pageserver( // Scan the local 'tenants/' directory and start loading the tenants BACKGROUND_RUNTIME.block_on(mgr::init_tenant_mgr(conf, remote_storage.clone()))?; - // shared state between the background task and the http endpoint; note that the http endpoint + // shared state between the disk-usage backed eviction background task and the http endpoint + // that allows triggering disk-usage based eviction manually. note that the http endpoint // is still accessible even if background task is not configured as long as remote storage has // been configured. let disk_usage_eviction_state: Arc = Arc::default(); diff --git a/pageserver/src/disk_usage_eviction_task.rs b/pageserver/src/disk_usage_eviction_task.rs index 38aeaf275a..e0083c11bb 100644 --- a/pageserver/src/disk_usage_eviction_task.rs +++ b/pageserver/src/disk_usage_eviction_task.rs @@ -72,7 +72,7 @@ pub struct DiskUsageEvictionTaskConfig { #[derive(Default)] pub struct DiskUsageEvictionState { - /// Exclude http requests and background task from running at the same time. + /// Prevent mgmt API requests and the background loop from running eviction at the same time. mutex: tokio::sync::Mutex<()>, }