Skip to content

Proto: migrate AggregateExec (function-codec path) #23512

Description

@adriangb

Part of #23494.

aggregates/mod.rs. Single child + PhysicalGroupBy + aggregate exprs. UDAFs go through ctx.encode_udaf / ctx.decode_udaf (store the returned Option<Vec<u8>> in fun_definition); AggregateMode by-name; orderings inline like SortExec. No raw codec.


Follow the pattern in #23494 (reference impl: #23495 / ProjectionExec; full end-state draft: #23496). Add try_to_proto inside impl ExecutionPlan for X, add the inherent X::try_from_proto, repoint the decode arm, and delete the old central arm in the same PR (that is the proof the hook is reached). Keep the wire format byte-for-byte identical.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions