[4.x] Fix how hash slot assignment is retrieved and stored

vert-x3 / vertx-redis-client

Redis client for Vert.x

http://vertx.io

Apache License 2.0

131 stars 119 forks source link

Closed Ladicek closed 1 year ago

Ladicek commented 1 year ago

Backport of #405 to 4.x.

Ladicek commented 1 year ago

The test failure is not caused by this PR, it fails on the current 4.x branch too (as well as in #406).

vietj commented 1 year ago

thanks @Ladicek let me fix the build and then you can rebase :-)

Ladicek commented 1 year ago

Refactored as discussed on Friday:

the hash slot assignment is not stored permanently; instead, it is treated as a cache with configurable TTL (defaults to 1 second)
avoided the storm of hash slot retrieval after the cache expired by storing Future<Slots> instead of simple Slots