Does Lettuce use CLUSTER NODES instead of CLUSTER SLOTS to discover topology? about lettuce HOT 3 OPEN

Seems like we are indeed using CLUSTER NODES, which currently is against the recommendation in the docs:

Note that normally clients willing to fetch the map between Cluster hash slots and node addresses should use CLUSTER SLOTS instead. CLUSTER NODES, that provides more information, should be used for administrative tasks, debugging, and configuration inspections. It is also used by redis-cli in order to manage a cluster.

This was initially implemented a while ago in #240 (Lettuce 4.2.0) and seems to have never been addressed since.

The change does not require new design, but has impact on large portions of the code.

Adding to the backlog until we have spare resources.

from lettuce.

I've been poking around a bit, it doesn't seem that risky, at least at first glance. What do you think about a new contributor (me) trying to tackle this?

Absolutely, feel free to submit a PR, you are more than welcome.

from lettuce.

I've been poking around a bit, it doesn't seem that risky, at least at first glance. What do you think about a new contributor (me) trying to tackle this?

from lettuce.