ExHammer / hammer-backend-redis

A Redis backend for for the Hammer rate-limiter
https://hexdocs.pm/ExHammer/hammer-backend-redis
MIT License
50 stars 32 forks source link

handle race condition where two nodes may attempt to create the same bucket simultaneously. #12

Closed mpinkston closed 6 years ago

mpinkston commented 6 years ago

Fixes #11

This PR is designed to handle a possible race condition where two (or more) nodes attempt to create the same bucket at the same time.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.1%) to 84.906% when pulling 03ca44207baf5e70e7dd5e852bfb6723741f462f on mpinkston:kaizen/multinode_race_condition into 40bb71515f35698bd821e3410bb1a14c3d52c51a on ExHammer:master.

johnnyshields commented 6 years ago

Add "Fixes #11" to the description so it is closed when merged.

JuneKelly commented 6 years ago

Sorry this slipped past me, my notifications are mis-configured. I'll get this merged ASAP.

JuneKelly commented 6 years ago

Released in 4.0.1, thanks everyone!