From 511978be6f5ba984890bb6f96c3338339760ead7 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 1b96362314..15adaaeb5e 100644 --- a/pageserver/src/tenant.rs +++ b/pageserver/src/tenant.rs @@ -1505,7 +1505,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. //