Fix build and CR comment

Author: fulmicoton-dd

quickwit/Cargo.lock

@@ -23,9 +23,15 @@ aws-smithy-types = { workspace = true }
 futures = { workspace = true }
 mini-moka = { workspace = true }
 tokio = { workspace = true }
+tracing = { workspace = true }
 
 quickwit-common = { workspace = true }
 
 [features]
 kinesis = ["aws-sdk-kinesis"]
 sqs = ["aws-sdk-sqs"]
+
+[package.metadata.cargo-machete]
+# tracing is only referenced from inside quickwit_common's rate_limited_* macros
+# (via an absolute `::tracing::` path), so it doesn't show up textually in this crate.
+ignored = ["tracing"]

quickwit/quickwit-aws/Cargo.toml

@@ -136,7 +136,7 @@ impl ResolveDns for CachingDnsResolver {
         let dns_entry_opt: Option<Arc<DnsEntry>> = cache.get(&host);
         // First insertion CAN trigger several DNS call, but this is not a problem.
         let dns_entry: Arc<DnsEntry> = if let Some(dns_entry) = dns_entry_opt {
-            dns_entry.clone()
+            dns_entry
         } else {
             let dns_entry = Arc::new(DnsEntry::new());
             cache.insert(host.clone(), dns_entry.clone());