Skip to content
This repository has been archived by the owner on Jul 19, 2023. It is now read-only.

Fix concurrent map access in ingester #763

Merged
merged 2 commits into from
Jun 9, 2023

Conversation

cyriltovena
Copy link
Collaborator

Introduced by faf62ed this was because before we were not flushing async and we would only sort/add profiles one at a time.

@korniltsev
Copy link
Collaborator

Wonder if we should add --race to the test runner

@cyriltovena
Copy link
Collaborator Author

So this has re-introduced a bit of latency not as much as before though. I'll investigate if there's a better way before mergning

@cyriltovena
Copy link
Collaborator Author

Latest commit bring back tail latency to 50ms see below

image

@cyriltovena cyriltovena merged commit b6d364b into main Jun 9, 2023
@cyriltovena cyriltovena deleted the fix-concurrent-map-access-ingester branch June 9, 2023 09:41
simonswine pushed a commit to simonswine/pyroscope that referenced this pull request Jun 30, 2023
* Fix concurrent map access in ingester

* Reduce the duration of the lock by only sweeping
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants