From 6fe3ffe498d3c33ce351ae15c425a0b957504489 Mon Sep 17 00:00:00 2001 From: Remi Dettai Date: Tue, 17 Jun 2025 16:07:55 +0200 Subject: [PATCH] Fix missing condition for scaling warning --- .../indexing_scheduler/scheduling/scheduling_logic.rs | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/quickwit/quickwit-control-plane/src/indexing_scheduler/scheduling/scheduling_logic.rs b/quickwit/quickwit-control-plane/src/indexing_scheduler/scheduling/scheduling_logic.rs index 1fdd597975c..4e31c3838b7 100644 --- a/quickwit/quickwit-control-plane/src/indexing_scheduler/scheduling/scheduling_logic.rs +++ b/quickwit/quickwit-control-plane/src/indexing_scheduler/scheduling/scheduling_logic.rs @@ -305,10 +305,12 @@ fn place_unassigned_shards_ignoring_affinity( match attempt_place_unassigned_shards(&unassigned_shards[..], &problem, partial_solution) { Ok(mut solution) => { // the higher the attempt number, the more unbalanced the solution - tracing::warn!( - attempt_number = attempt_number, - "capacity re-scaled, scheduling solution likely unbalanced" - ); + if attempt_number > 0 { + tracing::warn!( + attempt_number = attempt_number, + "capacity re-scaled, scheduling solution likely unbalanced" + ); + } solution.capacity_scaling_iterations = attempt_number; return solution; }