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

Consider duplicate stacktraces in custom index #102292

Merged

Conversation

danielmitterdorfer
Copy link
Member

With this commit we consider also duplicate stacktraces in custom indices for profiling events. We achieve that by using the recently introduced counted_terms aggregation (requires that the respective field is mapped as counted_keyword).

Relates #101826
Relates #102020

With this commit we consider also duplicate stacktraces in custom
indices for profiling events. We achieve that by using the recently
introduced `counted_terms` aggregation (requires that the respective
field is mapped as `counted_keyword`).

Relates elastic#101826
Relates elastic#102020
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/profiling (Team:Universal Profiling)

Copy link
Contributor

@rockdaboot rockdaboot 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 ca3b6f0 into elastic:main Nov 20, 2023
14 checks passed
@danielmitterdorfer danielmitterdorfer deleted the profiling-counted-terms branch November 20, 2023 07:05
andreidan pushed a commit to andreidan/elasticsearch that referenced this pull request Nov 22, 2023
With this commit we consider also duplicate stacktraces in custom
indices for profiling events. We achieve that by using the recently
introduced `counted_terms` aggregation (requires that the respective
field is mapped as `counted_keyword`).

Relates elastic#101826
Relates elastic#102020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>enhancement :UniversalProfiling/Application Elastic Universal Profiling REST APIs and infrastructure v8.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants