From b61f4b966f616ed3d231857403946149520aad2a Mon Sep 17 00:00:00 2001 From: Simon Beal <5381483+muddyfish@users.noreply.github.com> Date: Tue, 19 Nov 2024 16:53:40 +0000 Subject: [PATCH] Express cache cleanup (#1142) ## Description of change Remove unneeded todo Relevant issues: https://github.com/awslabs/mountpoint-s3/pull/1141#discussion_r1846841259 ## Does this change impact existing behavior? No ## Does this change need a changelog entry in any of the crates? No --- By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and I agree to the terms of the [Developer Certificate of Origin (DCO)](https://developercertificate.org/). Signed-off-by: Simon Beal --- mountpoint-s3/src/data_cache/express_data_cache.rs | 2 -- 1 file changed, 2 deletions(-) diff --git a/mountpoint-s3/src/data_cache/express_data_cache.rs b/mountpoint-s3/src/data_cache/express_data_cache.rs index 46ce485e4..7be14aa3b 100644 --- a/mountpoint-s3/src/data_cache/express_data_cache.rs +++ b/mountpoint-s3/src/data_cache/express_data_cache.rs @@ -62,8 +62,6 @@ where Client: ObjectClient + Send + Sync + 'static, { /// Create a new instance. - /// - /// TODO: consider adding some validation of the bucket. pub fn new(client: Client, config: ExpressDataCacheConfig, source_bucket_name: &str, bucket_name: &str) -> Self { Self { client,