Skip to content

compute: Emit FlatMap output as the columnar edge - #37768

Draft
antiguru wants to merge 1 commit into
columnar-p2-constantfrom
columnar-p4-flatmap-output
Draft

compute: Emit FlatMap output as the columnar edge#37768
antiguru wants to merge 1 commit into
columnar-p2-constantfrom
columnar-p4-flatmap-output

Conversation

@antiguru

@antiguru antiguru commented Jul 21, 2026

Copy link
Copy Markdown
Member

FlatMap emits its output as the columnar edge.

Columnar dataflow-edge migration. Design doc: doc/developer/design/20260720_columnar_dataflow_edges.md (#37744).

Part of CPU-51.

@linear-code

linear-code Bot commented Jul 22, 2026

Copy link
Copy Markdown

CPU-51

@antiguru
antiguru force-pushed the columnar-p4-flatmap-output branch from c14d129 to baad56b Compare July 22, 2026 16:24
@antiguru
antiguru force-pushed the columnar-p4-flatmap-output branch from baad56b to fa39476 Compare July 22, 2026 17:50
@antiguru
antiguru force-pushed the columnar-p4-flatmap-output branch from fa39476 to 242f5c0 Compare August 19, 2026 12:01
@antiguru
antiguru force-pushed the columnar-p4-flatmap-output branch 2 times, most recently from 4113b07 to 57e594c Compare August 20, 2026 08:49
@antiguru
antiguru force-pushed the columnar-p4-flatmap-output branch from 57e594c to 94d99b4 Compare August 20, 2026 09:13
Flip FlatMap's output to build into a `ConsolidatingColumnBuilder`, returning a
`CollectionEdge::Columnar`. The pre-migration ok builder was already a
consolidating one, so within-batch folding is preserved. Output rows are freshly
built by the mfp, so the owned give into staging is a move, not a new alloc.

The err collection stays `Vec`. The input side and fuel machinery are unchanged.

Adds a within-batch duplicate-fold test and updates the arms-agree and fuel tests
to decode the columnar capture.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@antiguru
antiguru force-pushed the columnar-p4-flatmap-output branch from 94d99b4 to f81c1a4 Compare September 6, 2026 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant