Skip to content

Commit 5d0b257

Browse files
antiguruclaude
andcommitted
doc: de-plan TopK map_topk_key comments
Comment-only, no behavior change. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 13c8b27 commit 5d0b257

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

‎src/compute/src/render/top_k.rs‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -161,9 +161,9 @@ impl<'scope, T: crate::render::RenderTimestamp + crate::render::MaybeBucketByTim
161161
let mut datum_vec = mz_repr::DatumVec::new();
162162
// A literal, non-negative limit skips this branch entirely, so this
163163
// per-row evaluation only runs for column or otherwise fallible
164-
// limits. On the `Vec` edge `into_vec` is the identity, so Wave 1 is
165-
// unchanged. On a columnar edge it is a narrow sanctioned decode
166-
// confined to this rare path.
164+
// limits. On the `Vec` edge `into_vec` is the identity, so the
165+
// `Vec` path is unchanged. On a columnar edge it is a narrow
166+
// sanctioned decode confined to this rare path.
167167
let errors = ok_input.clone().into_vec().flat_map(move |row| {
168168
let temp_storage = mz_repr::RowArena::new();
169169
let datums = datum_vec.borrow_with(&row);
@@ -622,8 +622,8 @@ impl<'scope, T: crate::render::RenderTimestamp + crate::render::MaybeBucketByTim
622622
/// avoid the per-record decode, because there is no columnar batcher to push
623623
/// borrowed rows into here. The `Vec` arm maps the collection directly and reuses
624624
/// the owned input row, so it is behaviorally identical to consuming the
625-
/// collection as before this migration. The columnar arm runs whenever the TopK
626-
/// input edge is columnar, which the upstream producers now emit.
625+
/// collection directly. The columnar arm runs whenever the TopK input edge is
626+
/// columnar.
627627
fn map_topk_key<'s, T, L>(
628628
edge: CollectionEdge<'s, T>,
629629
name: &str,

0 commit comments

Comments
 (0)