Skip to content

Commit fc429ab

Browse files
antiguruclaude
andcommitted
doc: de-plan concat_many comments
Comment-only, no behavior change. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent f541be1 commit fc429ab

1 file changed

Lines changed: 3 additions & 5 deletions

File tree

src/compute/src/render/columnar.rs

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -127,17 +127,15 @@ impl<'scope, T: RenderTimestamp> CollectionEdge<'scope, T> {
127127

128128
/// Concatenates a collection of edges.
129129
///
130-
/// Every producer emits the columnar variant, so the inputs concatenate
131-
/// natively into the columnar variant.
130+
/// The inputs are all columnar, so they concatenate natively into the
131+
/// columnar variant.
132132
pub fn concat_many<I>(scope: Scope<'scope, T>, edges: I) -> Self
133133
where
134134
I: IntoIterator<Item = Self>,
135135
{
136136
let cols = edges.into_iter().map(|edge| match edge {
137137
CollectionEdge::Columnar(c) => c,
138-
// No producer emits `Vec` after the migration, so a `Vec` input
139-
// cannot reach here. The `Vec` arm and this `unreachable!` are
140-
// removed together when the enum collapses to a columnar alias.
138+
// No producer emits `Vec`, so a `Vec` input cannot reach here.
141139
CollectionEdge::Vec(_) => unreachable!("no producer emits a `Vec` edge"),
142140
});
143141
CollectionEdge::Columnar(differential_dataflow::collection::concatenate(

0 commit comments

Comments
 (0)