From 8b44003647314a8be232888b6a494c9677cfa65d Mon Sep 17 00:00:00 2001 From: Vlad Lazar Date: Fri, 9 Feb 2024 14:54:14 +0000 Subject: [PATCH] review: validate vectored get in release mode too --- pageserver/src/tenant/timeline.rs | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/pageserver/src/tenant/timeline.rs b/pageserver/src/tenant/timeline.rs index e29d1bdf95..e7b7e1b914 100644 --- a/pageserver/src/tenant/timeline.rs +++ b/pageserver/src/tenant/timeline.rs @@ -758,10 +758,8 @@ impl Timeline { GetVectoredImpl::Vectored => { let vectored_res = self.get_vectored_impl(keyspace.clone(), lsn, ctx).await; - if cfg!(debug_assertions) { - self.validate_get_vectored_impl(&vectored_res, keyspace, lsn, ctx) - .await; - } + self.validate_get_vectored_impl(&vectored_res, keyspace, lsn, ctx) + .await; vectored_res } @@ -828,7 +826,6 @@ impl Timeline { Ok(results) } - #[allow(unused)] pub(super) async fn validate_get_vectored_impl( &self, vectored_res: &Result>, GetVectoredError>,