From e1ba2f96ab00c69662648f04fc10dc0f237acac0 Mon Sep 17 00:00:00 2001 From: Christian Schwarz Date: Mon, 27 Mar 2023 19:07:34 +0200 Subject: [PATCH] fix: post-merge, file_size() returns u64 --- pageserver/src/tenant/timeline.rs | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/pageserver/src/tenant/timeline.rs b/pageserver/src/tenant/timeline.rs index 1cd9709c21..2433567316 100644 --- a/pageserver/src/tenant/timeline.rs +++ b/pageserver/src/tenant/timeline.rs @@ -4065,9 +4065,7 @@ impl std::fmt::Debug for LocalLayerInfoForDiskUsageEviction { impl LocalLayerInfoForDiskUsageEviction { pub fn file_size(&self) -> u64 { - self.layer - .file_size() - .expect("we know this is a local layer") + self.layer.file_size() } } @@ -4079,7 +4077,7 @@ impl Timeline { let mut resident_layers = Vec::new(); for l in layers.iter_historic_layers() { - max_layer_size = max_layer_size.max(l.file_size()); + max_layer_size = max_layer_size.max(Some(l.file_size())); if l.is_remote_layer() { continue;