From 3c3b9dc9197dc43cce8bd24dfe4feab179ac118c Mon Sep 17 00:00:00 2001 From: Erik Grinaker Date: Thu, 13 Mar 2025 17:28:21 +0100 Subject: [PATCH] pageserver: enable `image_creation_preempt_threshold` by default (#11216) ## Problem This is already set in production, we should harmonize the default. ## Summary of changes Default `image_creation_preempt_threshold` to 3. --- libs/pageserver_api/src/config.rs | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/libs/pageserver_api/src/config.rs b/libs/pageserver_api/src/config.rs index 6e457823dd..c7d33d8a04 100644 --- a/libs/pageserver_api/src/config.rs +++ b/libs/pageserver_api/src/config.rs @@ -587,9 +587,8 @@ pub mod tenant_conf_defaults { pub const DEFAULT_GC_PERIOD: &str = "1 hr"; pub const DEFAULT_IMAGE_CREATION_THRESHOLD: usize = 3; // If there are more than threshold * compaction_threshold (that is 3 * 10 in the default config) L0 layers, image - // layer creation will end immediately. Set to 0 to disable. The target default will be 3 once we - // want to enable this feature. - pub const DEFAULT_IMAGE_CREATION_PREEMPT_THRESHOLD: usize = 0; + // layer creation will end immediately. Set to 0 to disable. + pub const DEFAULT_IMAGE_CREATION_PREEMPT_THRESHOLD: usize = 3; pub const DEFAULT_PITR_INTERVAL: &str = "7 days"; pub const DEFAULT_WALRECEIVER_CONNECT_TIMEOUT: &str = "10 seconds"; pub const DEFAULT_WALRECEIVER_LAGGING_WAL_TIMEOUT: &str = "10 seconds";