Skip to content

Conversation

@not-napoleon
Copy link
Member

Relates to #26220

Our composite coverage isn't bad, but I didn't see any rest tests for histograms. This PR adds them.

@elasticsearchmachine elasticsearchmachine added v8.8.0 needs:triage Requires assignment of a team area label labels Mar 24, 2023
@not-napoleon
Copy link
Member Author

@elasticmachine update branch

@not-napoleon not-napoleon added >non-issue >test Issues or PRs that are addressing/adding tests :Analytics/Aggregations Aggregations and removed needs:triage Requires assignment of a team area label labels Mar 28, 2023
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Mar 28, 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

@not-napoleon
Copy link
Member Author

@elasticmachine update branch

@not-napoleon
Copy link
Member Author

@elasticmachine update branch

@not-napoleon not-napoleon merged commit 85cc8f5 into elastic:main Apr 21, 2023
@not-napoleon not-napoleon deleted the composite-rest-tests branch April 21, 2023 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants