-
Notifications
You must be signed in to change notification settings - Fork 3.1k
[receiver/datadogreceiver] Use thread safe LRU #42883
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
base: main
Are you sure you want to change the base?
Conversation
|
|
Welcome, contributor! Thank you for your contribution to opentelemetry-collector-contrib. Important reminders:
A maintainer will review your pull request soon. Thank you for helping make OpenTelemetry better! |
|
Please review the CLA and sign it, as well as add a changelog to this PR, and mark it ready for review again. |
|
Is there a plan to fix this issue?My publishing process is somewhat blocked. Could you please help expedite the process? |
|
Pinging on this again as it fixes a fatal error. It seems the only blockers are the CLA and changelog. If the original contributor @rbeckmancostar isn't available, could a maintainer like @atoulme or @xiu please pick this up to finalize it? The code change itself is already approved. |
|
Sorry, I'm stuck with my legal dept w.r.t. the CLA and I'm not sure how long it will be until I get that resolved; if anyone else can progress this without my contribution/commit/anything related to me, please do. |
|
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
|
We are facing the same issue as well, hopefully this can be fixed as soon as possible! |
Description
Uses the thread safe lru package in golang-lru rather than the simplelru within that package.
Link to tracking issue
Fixes #42644
Potentially may fix the 40557 issue as well but I was not able to recreate it.
Testing
Performed the testing as directed in contributing.md