fix(traces): Drop aggregated trace data on API key failure to reduce memory usage by litianningdatadog · Pull Request #787 · DataDog/datadog-lambda-extension

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

@litianningdatadog

Copy link

Contributor

@litianningdatadog litianningdatadog commented

Aug 15, 2025

edited

Loading

In case of API key failure, we should drop the aggregated tracing data as there is no way to send them back. By doing so we expect to reduce the memory wasted on the unsent data.
Tested with fullinstrument stack in self-monitor repo. Manifest. Below chart shows the memory usage reduced with the proposed change.
image

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

August 15, 2025 18:35
…datadog-lambda-extension into tianning.li/purge-trace-flusher
…datadog-lambda-extension into tianning.li/purge-trace-flusher
…datadog-lambda-extension into tianning.li/purge-trace-flusher

@duncanista duncanista changed the title fix: Drop aggregated trace data on API key failure to reduce memory usage fix(traces): Drop aggregated trace data on API key failure to reduce memory usage

Aug 15, 2025

@litianningdatadog litianningdatadog merged commit 41695c4 into main

Aug 18, 2025

46 checks passed

@litianningdatadog litianningdatadog deleted the tianning.li/purge-trace-flusher branch

August 18, 2025 13:15

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

Reviewers

@duncanista duncanista duncanista approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@litianningdatadog @duncanista