Skip to content

[threadstats]fix threadstats thread safety + tests#6

Merged
yannmh merged 2 commits intomasterfrom
yann/test-thread-safety
Mar 31, 2015
Merged

[threadstats]fix threadstats thread safety + tests#6
yannmh merged 2 commits intomasterfrom
yann/test-thread-safety

Conversation

@yannmh
Copy link
Copy Markdown

@yannmh yannmh commented Mar 19, 2015

  • Fixes threadstats unsafe thread operations
  • Add tests to check statsd and threadstats thread safety.

See #5.

Thanks @sebest

@yannmh yannmh self-assigned this Mar 19, 2015
@yannmh yannmh force-pushed the yann/test-thread-safety branch from e0bc36a to 8c958e2 Compare March 19, 2015 22:29
@sebest
Copy link
Copy Markdown

sebest commented Mar 31, 2015

Hi, any reason why this pull request has not been merged yet?

thanx

@LeoCavaille
Copy link
Copy Markdown
Member

@yannmh can you rebase this PR?
The last version I checked out looked good.

@yannmh yannmh force-pushed the yann/test-thread-safety branch from 8c958e2 to ebfbd5b Compare March 31, 2015 14:02
yannmh pushed a commit that referenced this pull request Mar 31, 2015
[threadstats]fix threadstats thread safety + tests
@yannmh yannmh merged commit 8a522d7 into master Mar 31, 2015
@LeoCavaille LeoCavaille deleted the yann/test-thread-safety branch March 31, 2015 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants