Skip to content

Commit

Permalink
trying with just found_typ
Browse files Browse the repository at this point in the history
  • Loading branch information
mgree committed Dec 19, 2024
1 parent 4c29c55 commit fce5acb
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions src/adapter/src/coord/peek.rs
Original file line number Diff line number Diff line change
Expand Up @@ -317,13 +317,11 @@ pub fn create_fast_path_plan<T: Timestamp>(
let mut mir = &*dataflow_plan.objects_to_build[0].plan.as_inner_mut();
if let Some((rows, found_typ)) = mir.as_const() {
// In the case of a constant, we can return the result now.
let recalc_typ = mir.typ();
assert_eq!(found_typ, &recalc_typ);
return Ok(Some(FastPathPlan::Constant(
rows.clone()
.map(|rows| rows.into_iter().map(|(row, diff)| (row, diff)).collect()),
// For best accuracy, we need to recalculate typ.
recalc_typ,
found_typ,
)));
} else {
// If there is a TopK that would be completely covered by the finishing, then jump
Expand Down

0 comments on commit fce5acb

Please sign in to comment.