search

gosquared / redis-clustr

Redis Cluster client for Node.js
MIT License
107 stars 25 forks source link

Fix client selection for keyless commands and those with movable keys #27

Closed jbt closed 6 years ago

jbt commented 6 years ago

Prompted by #25 and #26 where we're incorrectly using the key interval being 0 to determine the target for a keyless command.

This PR includes an annoying amount of diff churn in config/commands.js but I've added some pre-sorting so it should be more stable next time we have to regenerate it.

jbt commented 6 years ago

Unfortunately the tests here aren't exactly comprehensive, but everything seems to be behaving and it's working with whatever I can throw at it, I'm going to merge this and get it into a 1.6.0 release