-
Notifications
You must be signed in to change notification settings - Fork 25.8k
Pipeline aggregations are weird #53742
Copy link
Copy link
Closed
Labels
Description
Pipeline aggregations are weird!
- They are validated on the data node rather than the coordinating node. (Move pipeline agg validation to coordinating node #53669 + More pipeline aggregation cleanup #54298)
- They are built on the data node and shipped back to the coordinating node (Save a little space in agg tree #53730 + Fix pipeline agg serialization for ccs #54282 + Begin to drop pipeline aggs from the result tree #54311 + More pipeline aggregation cleanup #54298)
- Because of that they implement
NamedWriteable,. None of that is really important if we stop doing the above. (Deprecate serializing PipelineAggregators #54926 + Remove serialization from pipeline aggregator #55026)equals, andhashcode
There are a few other things we could probably clean up here but this'd be a huge step for making pipeline aggregations simpler to implement. And, bonus, it'd save some space in the aggregation result tree.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Type
Fields
Give feedbackNo fields configured for issues without a type.