search

Workiva / go-datastructures

A collection of useful, performant, and threadsafe Go datastructures.
Apache License 2.0
7.66k stars 834 forks source link

RM-30900 Handle duplicated entry in lNodes for the concurrent trie #195

Closed stephane-martin closed 6 years ago

aviary2-wf commented 6 years ago

Security Insights

No security relevant content was detected by automated scans.

Action Items

dustinhiatt-wf commented 6 years ago

@stephane-martin can you reproduce this in a test to confirm the fix?

stephane-martin commented 6 years ago

@dustincataldo-wf see https://github.com/Workiva/go-datastructures/issues/194

dzyp commented 6 years ago

@stephane-martin

I added unit tests here: https://github.com/Workiva/go-datastructures/pull/198 that basically just use your example code. If you'd rather, feel free to pull those commits over to this PR and I'll close the other one.

stephane-martin commented 6 years ago

Thanks :-)

Le 1 juillet 2018 23:18:42 Dustin Hiatt notifications@github.com a écrit :

@stephane-martin I added unit tests here: #198 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.