Redis Cluster slot ranges

[travisbell]

I noticed a very small percentage of my traffic was throwing an error, and I was confused as to why until I saw the code below. It's a subtle bug in the code that calculates the slot ranges:

async-redis/lib/async/redis/cluster_client.rb

Line 109 in 6a75f89

range = Range.new(*slots, exclude_end: false)

With exclude_end defined, the ranges are not including their last values.

Range.new(*[12288, 16383], exclude_end: false)
=> 12288...16383

Whereas, we want to include the last values:

Range.new(*[12288, 16383])
=> 12288..16383

This was causing the last slot for every node to not be found using RangeMap#find.

[ioquatix]

Sorry about this, that looks like a bug!

[ioquatix]

Fix released, please let me know how you get on.