Skip to content
This repository was archived by the owner on Aug 23, 2023. It is now read-only.

fix issue 946 #949

Merged
merged 2 commits into from
Jul 30, 2018
Merged

fix issue 946 #949

merged 2 commits into from
Jul 30, 2018

Conversation

replay
Copy link
Contributor

@replay replay commented Jun 22, 2018

This should fix #946

This issue is very hard to reproduce, so I can't really test if this fixes the problem. I did test find tag queries, and they still work.

Can't add @shanson7 as reviewer, so I name-tag here

@replay replay requested a review from Dieterbe June 22, 2018 10:04
shanson7 added a commit to bloomberg/metrictank that referenced this pull request Jul 20, 2018
Copy link
Collaborator

@shanson7 shanson7 left a comment

Choose a reason for hiding this comment

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

LGTM!

@replay replay merged commit 3ce976f into master Jul 30, 2018
@Dieterbe Dieterbe deleted the fix_946 branch September 18, 2018 08:58
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.

Race condition in FindTags
2 participants