From 46b6a1a5e8b5ca209fa06dc79ff9ca7642a07a03 Mon Sep 17 00:00:00 2001 From: Joonas Koivunen Date: Thu, 24 Aug 2023 18:35:35 +0300 Subject: [PATCH] review comment: xref tested string --- pageserver/src/tenant/remote_timeline_client/upload.rs | 2 ++ 1 file changed, 2 insertions(+) diff --git a/pageserver/src/tenant/remote_timeline_client/upload.rs b/pageserver/src/tenant/remote_timeline_client/upload.rs index a805e9bd60..b53a760f75 100644 --- a/pageserver/src/tenant/remote_timeline_client/upload.rs +++ b/pageserver/src/tenant/remote_timeline_client/upload.rs @@ -67,6 +67,8 @@ pub(super) async fn upload_timeline_layer<'a>( // upload. However, a nonexistent file can also be indicative of // something worse, like when a file is scheduled for upload before // it has been written to disk yet. + // + // This is tested against `test_compaction_delete_before_upload` info!(path = %source_path.display(), "File to upload doesn't exist. Likely the file has been deleted and an upload is not required any more."); return Ok(()); }