From 51da078dc83ed5c12cf858605d2e2cdca533930e Mon Sep 17 00:00:00 2001 From: discord9 Date: Fri, 23 Oct 2026 15:23:54 +0800 Subject: [PATCH] chore(query): clear the two lints clippy reports on this branch A doc comment on the categorizer was followed by a blank line, and the test table builder takes eight arguments. Both are what CI's `-D warnings` turns into errors; neither changes behaviour. Signed-off-by: discord9 --- src/query/src/dist_plan/commutativity.rs | 1 - src/query/src/promql/planner/test.rs | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) 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) -> DfTableSou .await } +#[allow(clippy::too_many_arguments)] async fn build_test_table_provider_with_options( table_name_tuples: &[(String, String)], num_tag: usize,