diff --git a/query-engine/connectors/sql-query-connector/src/sql_trace.rs b/query-engine/connectors/sql-query-connector/src/sql_trace.rs
index 17965ab21ee2..b5e0643b2127 100644
--- a/query-engine/connectors/sql-query-connector/src/sql_trace.rs
+++ b/query-engine/connectors/sql-query-connector/src/sql_trace.rs
@@ -33,7 +33,6 @@ macro_rules! sql_trace {
             // Temporary method to pass the traceid in an operation
             fn add_trace_id(self, trace_id: Option<String>) -> Self {
                 if let Some(traceparent) = trace_id {
-                    let out = should_sample(&traceparent);
                     if should_sample(&traceparent) {
                         self.comment(format!("traceparent={}", traceparent))
                     } else {
@@ -48,7 +47,7 @@ macro_rules! sql_trace {
 }
 
 fn should_sample(traceparent: &str) -> bool {
-    let trace_info: Vec<&str> = traceparent.split("-").collect();
+    let trace_info: Vec<&str> = traceparent.split('-').collect();
 
     if trace_info.len() != 4 {
         false