search

Ingensi / dockbeat

Dockbeat - the elastic Beat for docker daemon monitoring
Other
286 stars 70 forks source link

Another proposal resolving concurrent map writes #100

Closed marminthibaut closed 8 years ago

marminthibaut commented 8 years ago

Here is another proposal to resolve the concurrent map writes bug (see #90).

@TheStigger : can you compile and run this version and see if it resolves the bug?

Thx

codecov-io commented 8 years ago

Current coverage is 66.07%

Merging #100 into develop will increase coverage by +0.46% as of c6a015e

@@            develop    #100   diff @@
=======================================
  Files             7       7       
  Stmts           442     454    +12
  Branches         28      28       
  Methods           0       0       
=======================================
+ Hit             290     300    +10
  Partial           7       7       
- Missed          145     147     +2

Review entire Coverage Diff as of c6a015e

Powered by Codecov. Updated on successful CI builds.

marminthibaut commented 8 years ago

What do you think about this PR @erwanncloarec ?

marminthibaut commented 8 years ago

It closes #90

Leave feedback if prob appears again :smile: