Skip to content
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

Support sampling in counted_terms aggregation #103846

Merged

Conversation

danielmitterdorfer
Copy link
Member

With this commit we support using the counted_terms aggregation in a sampling context (e.g. as a sub-aggregation in a random_sampler aggregation).

With this commit we support using the `counted_terms` aggregation in a
sampling context (e.g. as a sub-aggregation in a `random_sampler`
aggregation).
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine
Copy link
Collaborator

Hi @danielmitterdorfer, I've created a changelog YAML for you.

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

@danielmitterdorfer danielmitterdorfer merged commit 59d0470 into elastic:main Jan 3, 2024
16 checks passed
@danielmitterdorfer danielmitterdorfer deleted the counted-keyword-sampling branch January 3, 2024 12:10
jbaiera pushed a commit to jbaiera/elasticsearch that referenced this pull request Jan 10, 2024
With this commit we support using the `counted_terms` aggregation in a
sampling context (e.g. as a sub-aggregation in a `random_sampler`
aggregation).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/Aggregations Aggregations >enhancement Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) test-update-serverless v8.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants