Fix cluster-slots discover race condition again

redis / node-redis

Redis Node.js client

https://redis.js.org/

MIT License

16.95k stars 1.89k forks source link

Fix cluster-slots discover race condition again #2867

Closed jjsimps closed 8 hours ago

jjsimps commented 1 week ago

Description

This re-applies the fix done in #2731, as noted by @edwin-jones.

See #2704 for details.

Checklist

[ ] Does npm test pass with this change (including linting)?
[ ] Is the new or changed code fully tested?
[ ] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

leibale commented 8 hours ago

Sorry about it.. I'm not sure what happened, but I'll need to make sure that we did not lose more work.. BTW, the latest non "next" release still contains this fix