diff --git a/pageserver/src/tenant/storage_layer.rs b/pageserver/src/tenant/storage_layer.rs index 4107161aa1..6385c4ac51 100644 --- a/pageserver/src/tenant/storage_layer.rs +++ b/pageserver/src/tenant/storage_layer.rs @@ -344,9 +344,6 @@ impl LayerAccessStats { /// are used in (timeline). #[async_trait::async_trait] pub trait Layer: std::fmt::Debug + std::fmt::Display + Send + Sync + 'static { - /// Range of keys that this layer covers - fn get_key_range(&self) -> Range; - /// Inclusive start bound of the LSN range that this layer holds /// Exclusive end bound of the LSN range that this layer holds. /// diff --git a/pageserver/src/tenant/storage_layer/delta_layer.rs b/pageserver/src/tenant/storage_layer/delta_layer.rs index f479687a2b..7512098fc5 100644 --- a/pageserver/src/tenant/storage_layer/delta_layer.rs +++ b/pageserver/src/tenant/storage_layer/delta_layer.rs @@ -255,10 +255,6 @@ impl Layer for DeltaLayer { .await } - fn get_key_range(&self) -> Range { - self.get_key_range() - } - fn get_lsn_range(&self) -> Range { self.get_lsn_range() } @@ -394,10 +390,6 @@ impl DeltaLayer { .await } - pub(crate) fn get_key_range(&self) -> Range { - self.layer_desc().key_range.clone() - } - pub(crate) fn get_lsn_range(&self) -> Range { self.layer_desc().lsn_range.clone() } diff --git a/pageserver/src/tenant/storage_layer/image_layer.rs b/pageserver/src/tenant/storage_layer/image_layer.rs index c144ce562d..069e0ebed2 100644 --- a/pageserver/src/tenant/storage_layer/image_layer.rs +++ b/pageserver/src/tenant/storage_layer/image_layer.rs @@ -186,11 +186,6 @@ impl Layer for ImageLayer { .await } - /// Boilerplate to implement the Layer trait, always use layer_desc for persistent layers. - fn get_key_range(&self) -> Range { - self.get_key_range() - } - /// Boilerplate to implement the Layer trait, always use layer_desc for persistent layers. fn get_lsn_range(&self) -> Range { self.get_lsn_range() @@ -288,10 +283,6 @@ impl ImageLayer { .with_context(|| format!("read {}", self.path().display())) } - pub(crate) fn get_key_range(&self) -> Range { - self.layer_desc().key_range.clone() - } - /// Boilerplate to implement the Layer trait, always use layer_desc for persistent layers. pub(crate) fn get_lsn_range(&self) -> Range { self.layer_desc().lsn_range.clone() diff --git a/pageserver/src/tenant/storage_layer/inmemory_layer.rs b/pageserver/src/tenant/storage_layer/inmemory_layer.rs index 0c85f867a7..95988252aa 100644 --- a/pageserver/src/tenant/storage_layer/inmemory_layer.rs +++ b/pageserver/src/tenant/storage_layer/inmemory_layer.rs @@ -107,10 +107,6 @@ impl InMemoryLayer { self.end_lsn.get().copied().unwrap_or(Lsn::MAX) } - pub(crate) fn get_key_range(&self) -> Range { - Key::MIN..Key::MAX - } - pub(crate) fn get_lsn_range(&self) -> Range { self.start_lsn..self.end_lsn_or_max() } @@ -222,10 +218,6 @@ impl InMemoryLayer { #[async_trait::async_trait] impl Layer for InMemoryLayer { - fn get_key_range(&self) -> Range { - self.get_key_range() - } - fn get_lsn_range(&self) -> Range { self.get_lsn_range() } diff --git a/pageserver/src/tenant/storage_layer/remote_layer.rs b/pageserver/src/tenant/storage_layer/remote_layer.rs index 610bbe7dc0..961c64f215 100644 --- a/pageserver/src/tenant/storage_layer/remote_layer.rs +++ b/pageserver/src/tenant/storage_layer/remote_layer.rs @@ -95,11 +95,6 @@ impl Layer for RemoteLayer { Ok(()) } - /// Boilerplate to implement the Layer trait, always use layer_desc for persistent layers. - fn get_key_range(&self) -> Range { - self.layer_desc().key_range.clone() - } - /// Boilerplate to implement the Layer trait, always use layer_desc for persistent layers. fn get_lsn_range(&self) -> Range { self.layer_desc().lsn_range.clone()