Fix tests

rdettai · rdettai · commit 1b57434183db · 2025-06-11T11:19:56.000+02:00
diff --git a/quickwit/quickwit-control-plane/src/indexing_scheduler/scheduling/mod.rs b/quickwit/quickwit-control-plane/src/indexing_scheduler/scheduling/mod.rs
@@ -536,7 +536,10 @@ fn assert_post_condition_physical_plan_match_solution(
     assert_eq!(num_indexers, id_to_ord_map.indexer_ids.len());
     let mut reconstructed_solution = SchedulingSolution::with_num_indexers(num_indexers);
     convert_physical_plan_to_solution(physical_plan, id_to_ord_map, &mut reconstructed_solution);
-    assert_eq!(solution, &reconstructed_solution);
+    assert_eq!(
+        solution.indexer_assignments,
+        reconstructed_solution.indexer_assignments
+    );
 }
 
 fn add_shard_to_indexer(
diff --git a/quickwit/quickwit-control-plane/src/indexing_scheduler/scheduling/scheduling_logic_model.rs b/quickwit/quickwit-control-plane/src/indexing_scheduler/scheduling/scheduling_logic_model.rs
@@ -229,7 +229,7 @@ impl IndexerAssignment {
     }
 }
 
-#[derive(Clone, Debug, Eq, PartialEq)]
+#[derive(Clone, Debug)]
 pub struct SchedulingSolution {
     pub indexer_assignments: Vec<IndexerAssignment>,
     // used for tests

Original file line number	Diff line number	Diff line change
`@@ -229,7 +229,7 @@ impl IndexerAssignment {`
`229`	`229`	`}`
`230`	`230`	`}`
`231`	`231`
`232`		`-#[derive(Clone, Debug, Eq, PartialEq)]`
	`232`	`+#[derive(Clone, Debug)]`
`233`	`233`	`pub struct SchedulingSolution {`
`234`	`234`	`pub indexer_assignments: Vec<IndexerAssignment>,`
`235`	`235`	`// used for tests`