Skip to content

Rest tests for Composite Histogarm #94753

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Apr 21, 2023

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