Skip to content

Commit 3f3c394

Browse files
zzylolclaude
andcommitted
fix: integrate with #604
#604 gave RootDemand a latency_ms and a logical candidate several physical candidates. The Hydra count test sets no latency bound and checks the all-query-time physical candidate, which comes first. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 parent 0b1a8c3 commit 3f3c394

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

‎crates/integration-tests/tests/pass1_sql_coverage.rs‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -191,6 +191,7 @@ async fn grouped_count_offers_a_priced_executable_hydra_plan() {
191191
execute_at: None,
192192
},
193193
predictability: Predictability::default(),
194+
latency_ms: None,
194195
}];
195196
let data = DataWorkload {
196197
arrival: DataArrival::ContinuouslyIngesting,
@@ -219,7 +220,8 @@ async fn grouped_count_offers_a_priced_executable_hydra_plan() {
219220
.collect();
220221
assert!(!hydra.is_empty(), "a Hydra candidate is generated");
221222
for (candidate, root) in hydra {
222-
let physical = candidate.physical.as_ref().expect("Stage 2 builds it");
223+
// The all-query-time physical candidate comes first (#604).
224+
let physical = candidate.physical.first().expect("Stage 2 builds it");
223225
assert!(
224226
enumeration.selection.costs.contains_key(&physical.id),
225227
"{} is priced: {:?}",

0 commit comments

Comments
 (0)