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

Fix race in go-metrics adapater #4098

Merged
merged 1 commit into from
Apr 24, 2017
Merged

Conversation

urso
Copy link

@urso urso commented Apr 24, 2017

Fix race in go-metrics adapter, if registry is updated concurrently from
multiple go routines.

@urso urso added bug libbeat needs_backport PR is waiting to be backported to other branches. review v5.3.2 labels Apr 24, 2017
@urso urso force-pushed the fix/go-metrics-race branch from 451297b to 8d0d8e1 Compare April 24, 2017 10:08
Copy link
Contributor

@tsg tsg left a comment

Choose a reason for hiding this comment

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

LGTM, waiting for green.

Fix race in go-metrics adapter, if registry is updated concurrently from
multiple go routines.
@urso urso force-pushed the fix/go-metrics-race branch from 8d0d8e1 to 7eaaec3 Compare April 24, 2017 11:34
@ruflin ruflin merged commit 38e04b4 into elastic:master Apr 24, 2017
@urso urso removed the needs_backport PR is waiting to be backported to other branches. label Apr 24, 2017
@tsg tsg added the needs_backport PR is waiting to be backported to other branches. label Apr 24, 2017
@tsg
Copy link
Contributor

tsg commented Apr 24, 2017

Can you also backport this to the 5.4 branch, please? I added the needs_backport label.

@urso urso removed the needs_backport PR is waiting to be backported to other branches. label Apr 24, 2017
@urso urso deleted the fix/go-metrics-race branch February 19, 2019 18:45
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.

3 participants