brimdata / super

An analytics database that puts JSON and relational tables on equal footing
https://zed.brimdata.io/
BSD 3-Clause "New" or "Revised" License
1.4k stars 67 forks source link

Fix vector summarize partials-in handling #5447

Closed nwt closed 2 weeks ago

nwt commented 2 weeks ago

vam/op/summarize.Summarize doesn't know how to extract partials values from an input vector when operating in partials-in mode. Add missing wiring for that. (The new vam/op/summarize.Summarize.aggExprs field is analogous to the existing sam/op/groupby.Aggregator.aggRefs field.)

This was an easy thing to miss in the initial partials implementation given that we don't have an easy way to test it.