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

Limit the number of exemplars to be added for each dimension set #29242

Merged
merged 4 commits into from
Dec 12, 2023

Conversation

aishyandapalli
Copy link
Contributor

Description: Fix Memory usage for span metrics by limiting the number of exemplars per unique dimensions set/metric key

Copy link
Contributor

@albertteoh albertteoh left a comment

Choose a reason for hiding this comment

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

Thanks @aishyandapalli! I've added a few suggestions.

connector/spanmetricsconnector/config.go Outdated Show resolved Hide resolved
connector/spanmetricsconnector/internal/metrics/metrics.go Outdated Show resolved Hide resolved
connector/spanmetricsconnector/internal/metrics/metrics.go Outdated Show resolved Hide resolved
@aishyandapalli aishyandapalli force-pushed the span-metrics-oom-fix branch 4 times, most recently from 9112a8f to 58c1c2f Compare November 30, 2023 21:39
@evan-bradley
Copy link
Contributor

@aishyandapalli It looks like some unrelated changes got attached to your branch. Could you take a look?

If practical, squashing all of your changes into a single commit and rebasing on main may help remove the extra commits.

@aishyandapalli aishyandapalli force-pushed the span-metrics-oom-fix branch 3 times, most recently from 07c7ee6 to 0a2a195 Compare December 5, 2023 21:42
@aishyandapalli
Copy link
Contributor Author

@evan-bradley Had some issue with the rebase. Should be fine now

@evan-bradley evan-bradley merged commit 5c2ef75 into open-telemetry:main Dec 12, 2023
83 checks passed
@github-actions github-actions bot added this to the next release milestone Dec 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants