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

fix issue 946#949

Merged
replay merged 2 commits intomasterfrom
fix_946
Jul 30, 2018
Merged

fix issue 946#949
replay merged 2 commits intomasterfrom
fix_946

Conversation

@replay
Copy link
Copy Markdown
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
Copy Markdown
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