diff --git a/src/query/src/dist_plan/commutativity.rs b/src/query/src/dist_plan/commutativity.rs
index 17114e5dc55..940f921c7cf 100644
--- a/src/query/src/dist_plan/commutativity.rs
+++ b/src/query/src/dist_plan/commutativity.rs
@@ -393,7 +393,6 @@ impl Categorizer {
 /// several series at once. Neither has a compiler producer yet, which is why refusing them costs
 /// nothing today: a tree built some other way is exactly what this check is for, because evaluating
 /// it one region at a time would fold a partial input and answer a wrong query silently.
-
 #[cfg(test)]
 mod tests {
     use std::collections::{BTreeMap, BTreeSet};
diff --git a/src/query/src/promql/planner/test.rs b/src/query/src/promql/planner/test.rs
index 01af409fca3..15732c987bb 100644
--- a/src/query/src/promql/planner/test.rs
+++ b/src/query/src/promql/planner/test.rs
@@ -970,6 +970,7 @@ async fn native_engine_provider(partition_key_indices: Vec<usize>) -> DfTableSou
     .await
 }
 
+#[allow(clippy::too_many_arguments)]
 async fn build_test_table_provider_with_options(
     table_name_tuples: &[(String, String)],
     num_tag: usize,