From a958febd7a84b898c8baff3f972d1a4f9b9f0384 Mon Sep 17 00:00:00 2001 From: Christian Schwarz Date: Tue, 21 Jan 2025 12:32:55 +0100 Subject: [PATCH] reference issue that will remote hard-coded sequential() --- pageserver/src/tenant/storage_layer.rs | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/pageserver/src/tenant/storage_layer.rs b/pageserver/src/tenant/storage_layer.rs index 5cfe8c115a..8669b8e5a9 100644 --- a/pageserver/src/tenant/storage_layer.rs +++ b/pageserver/src/tenant/storage_layer.rs @@ -237,9 +237,13 @@ impl std::fmt::Debug for SelectedIoConcurrency { } impl IoConcurrency { - /// TODO: over time, work towards removing this method. - /// Requires finding a long-lived root for the IoConcurrency and funneling it through - /// to the place that does the get_values_reconstruct_data call. + /// Force sequential IO. This is a temporary workaround until we have + /// moved plumbing-through-the-call-stack + /// of IoConcurrency into `RequestContextq. + /// + /// DO NOT USE for new code. + /// + /// Tracking issue: . pub(crate) fn sequential() -> Self { Self::spawn(SelectedIoConcurrency::Sequential) }