You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The missing for TopMetricsAggregation is defined per field and not per aggregation. Currently, it is only possible to define the field but no missing property.
🐛 Wrong type
The
missing
forTopMetricsAggregation
is defined per field and not per aggregation. Currently, it is only possible to define thefield
but nomissing
property.elasticsearch-specification/specification/_types/aggregations/metric.ts
Line 431 in c7ee66d
see https://www.elastic.co/docs/reference/aggregations/search-aggregations-metrics-top-metrics#_missing
Besides that,
TopMetricsAggregation
inherits fromMetricAggregationBase
which wrongly inherits amissing
andscript
, which are both not available according to https://www.elastic.co/docs/reference/aggregations/search-aggregations-metrics-top-metrics . Note, themissing
is wrongly available per aggregation and not per field.Definition
As a first solution to at least provide support for a proper definition of
missing
, theTopMetricsValue
needs to be extended bymissing: FieldValue
:Changing the inheritance from
MetricAggregationBase
might have further implications I'm not aware of.The text was updated successfully, but these errors were encountered: