Don't send OutOfMemoryError to the telemetry by amarziali · Pull Request #10211 · DataDog/dd-trace-java

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@amarziali

Copy link

Contributor

What Does This Do

There is no big added value to send OOM stacktraces to the telemetry, since this is not directly related to our code's responsability .

Motivation

Additional Notes

Contributor Checklist

Jira ticket: [PROJ-IDENT]

@amarziali amarziali added the type: enhancement

Enhancements and improvements

label

Dec 15, 2025

@amarziali amarziali requested a review from a team as a code owner

December 15, 2025 09:18

@amarziali amarziali added the comp: telemetry

Telemetry

label

Dec 15, 2025

@amarziali amarziali requested a review from ygree

December 15, 2025 09:18
Copy link

Contributor

@mcculls mcculls left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense - they're mostly noise due to the lack of context

@amarziali amarziali enabled auto-merge (squash)

December 15, 2025 10:29

@amarziali amarziali merged commit 1f4c004 into master

Dec 15, 2025

565 of 566 checks passed

@amarziali amarziali deleted the andrea.marziali/exclude-oom branch

December 15, 2025 10:37

@github-actions github-actions bot added this to the 1.57.0 milestone

Dec 15, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mcculls mcculls mcculls approved these changes

@ygree ygree Awaiting requested review from ygree ygree is a code owner automatically assigned from DataDog/apm-java

Assignees

No one assigned

Labels

comp: telemetry

Telemetry

type: enhancement

Enhancements and improvements

Projects

None yet

Milestone

1.57.0

Development

Successfully merging this pull request may close these issues.

2 participants

@amarziali @mcculls