Skip to content

Commit

Permalink
Express cache cleanup (#1142)
Browse files Browse the repository at this point in the history
## Description of change

Remove unneeded todo

Relevant issues:
#1141 (comment)

## 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 <[email protected]>
  • Loading branch information
muddyfish authored Nov 19, 2024
1 parent 02f8dda commit b61f4b9
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions mountpoint-s3/src/data_cache/express_data_cache.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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,
Expand Down

0 comments on commit b61f4b9

Please sign in to comment.