Skip to content

Commit 4e3f1d8

Browse files
pquentingithub-actions[bot]
authored andcommitted
Mark GeoLineAggregation sort as optional (#5551)
While it's required most of the time, it's optional when in a sub-aggregation inside a time_series aggregation, as mentioned in the comment. (cherry picked from commit c5e3b91)
1 parent 1d4757f commit 4e3f1d8

File tree

5 files changed

+5
-7
lines changed

5 files changed

+5
-7
lines changed

output/openapi/elasticsearch-openapi.json

Lines changed: 1 addition & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

output/openapi/elasticsearch-serverless-openapi.json

Lines changed: 1 addition & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

output/schema/schema.json

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

output/typescript/types.ts

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

specification/_types/aggregations/metric.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,7 @@ export class GeoLineAggregation {
140140
* The name of the numeric field to use as the sort key for ordering the points.
141141
* When the `geo_line` aggregation is nested inside a `time_series` aggregation, this field defaults to `@timestamp`, and any other value will result in error.
142142
*/
143-
sort: GeoLineSort
143+
sort?: GeoLineSort
144144
/**
145145
* When `true`, returns an additional array of the sort values in the feature properties.
146146
*/

0 commit comments

Comments
 (0)