From 8e51d5d8f315e26f57004edd7e49de9a45cdcc7e Mon Sep 17 00:00:00 2001 From: Joonas Koivunen Date: Wed, 10 May 2023 17:35:26 +0300 Subject: [PATCH] doc: remove FIXME simplify uploading there's no need to simplify the uploading; the uploading is not on within this one future which will be executed so it's not safe to just remove it or revert it back. also I am hesitant to add new changes at this point, the diff is large enough. --- pageserver/src/tenant.rs | 1 - 1 file changed, 1 deletion(-) diff --git a/pageserver/src/tenant.rs b/pageserver/src/tenant.rs index 1c8454e27e..553e70c947 100644 --- a/pageserver/src/tenant.rs +++ b/pageserver/src/tenant.rs @@ -1440,7 +1440,6 @@ impl Tenant { let span = tracing::Span::current(); - // FIXME: simplify uploading // if we have concurrent requests, we will only execute one version of following future; // initially it did not have any means to be cancelled. //