Closed leeym closed 5 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 80.05%. Comparing base (
6804c15
) to head (d85f4f2
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I don't have write access to this repository so please merge the PR for me. Thanks.
Great! Thanks!
This PR fixes two issues
redis.Client
withredis.UniversalClient
. If we do use cluster client and if the cluster is large enough, it is possible that when accessing multiple keys in leaky bucket or token bucket, these keys might go different slots and it will fail with error messageCROSSSLOT Keys in request don't hash to the same slot
. Adding hash tags inredisKey
will force them into the same slot for keys with the same prefix.