Skip to content

Address some correlation id generator issues

Mikhail Mazurskiy requested to merge ash2k/generator-issues into master

I noticed that in !85 (merged) I haven't wrapped the Read() method in a mutex and decided to address that. It turned you we import and use both v1 and v2 ulid package, so I dropped v1 by updating the import in tests. Also added tests for the thread-safe wrapper and enabled race detector for tests. There are three individual commits. I hope it's ok to have all of them in a single MR. Creating 3 MRs feels like an overkill here.

Edited by Mikhail Mazurskiy

Merge request reports