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.

Sign up for GitHub

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

Jump to bottom

Grouping key: Use log.message or exception.message if there's nothing else #402

Closed
roncohen opened this issue Dec 14, 2017 · 0 comments · Fixed by #435
Closed

Grouping key: Use log.message or exception.message if there's nothing else #402

roncohen opened this issue Dec 14, 2017 · 0 comments · Fixed by #435
Assignees
@graphaelli

Comments

@roncohen
Copy link
Contributor

There are edge cases in which agents will send up a log.message or exception.message but no stack trace or param_message. In these cases, APM Server should take message into account when calculating the grouping key.
@graphaelli graphaelli self-assigned this Dec 22, 2017
graphaelli added a commit to graphaelli/apm-server that referenced this issue Dec 22, 2017
@graphaelli
consider exception or log message for grouping key if nothing else is…
963ee22 
… available

closes elastic#402
@graphaelli graphaelli mentioned this issue Dec 22, 2017
Merged
simitt pushed a commit that referenced this issue Jan 2, 2018
@graphaelli @simitt
consider exception or log message for grouping key if nothing else is…
c066b58 
… available (#435)

closes #402
simitt pushed a commit to simitt/apm-server that referenced this issue Jan 8, 2018
@graphaelli @simitt
consider exception or log message for grouping key if nothing else is…
a96081b 
… available (elastic#435)

closes elastic#402
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@graphaelli graphaelli

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

consider exception or log message for grouping key graphaelli/apm-server
2 participants
@roncohen @graphaelli