m-lab / go

General purpose libraries / APIs for use in mlab code.
Apache License 2.0
5 stars 6 forks source link

use ticker for simpler and faster logEvery #141

Closed gfr10598 closed 2 years ago

gfr10598 commented 2 years ago

The code I wrote a couple years ago is way more complex than necessary, and doesn't perform very well to boot.

This is much more idiomatic go, is about 4 times faster, and fewer lines of simpler code.


This change is Reviewable

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 1081


Totals Coverage Status
Change from base Build 1053: -0.006%
Covered Lines: 1993
Relevant Lines: 2120

💛 - Coveralls