Skip to content

Conversation

kkrik-es
Copy link
Contributor

The added tests cover the documented functionality.

Related to #26220.

@kkrik-es kkrik-es added >test Issues or PRs that are addressing/adding tests :Analytics/Aggregations Aggregations Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Apr 27, 2023
@kkrik-es kkrik-es self-assigned this Apr 27, 2023
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kkrik-es kkrik-es merged commit e05c6b1 into elastic:main May 2, 2023
@kkrik-es kkrik-es deleted the fix/26220 branch May 2, 2023 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/Aggregations Aggregations Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v8.9.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants