Skip to content

Commit f459648

Browse files
antiguruclaude
andcommitted
test: exercise concat_many with all-columnar inputs
The concat_many teardown made the Vec arm unreachable, so a mixed Vec/Columnar input can no longer reach concat_many. Feed two columnar edges instead, which is the only shape producers now emit. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent f72a571 commit f459648

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

‎src/compute/src/render/columnar.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -523,7 +523,7 @@ mod tests {
523523
}
524524

525525
#[mz_ore::test]
526-
fn concat_many_mixed_upgrades_to_columnar() {
526+
fn concat_many_concatenates_columnar() {
527527
let rows = test_rows();
528528
let expected: Vec<_> = {
529529
let mut updates: Vec<_> = rows
@@ -542,7 +542,7 @@ mod tests {
542542
let edge = CollectionEdge::concat_many(
543543
scope,
544544
[
545-
CollectionEdge::Vec(collection1),
545+
CollectionEdge::Columnar(vec_to_columnar(collection1)),
546546
CollectionEdge::Columnar(vec_to_columnar(collection2)),
547547
],
548548
);

0 commit comments

Comments
 (0)