envoyproxy/envoy

Change Redis cluster discovery to use CLUSTER_SHARDS API

Ouverte

#36 975 ouverte le 4 nov. 2024

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (0 personne assignée)C++ (5 373 forks)batch import
area/redisenhancementhelp wanted

Métriques du dépôt

Stars
 (27 997 étoiles)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

Title: Change Redis cluster discovery to use CLUSTER_SHARDS API

Description: Currently Envoy uses CLUSTER_SLOTS to fetch cluster topology from Redis. CLUSTER_SLOTS is deprecated in Redis 7.0. It is slower and less efficient than the new API, CLUSTER_SHARDS. We observed CLUSTER_SHARDS payload size to be 10x smaller than the CLUSTER_SLOTS payload for big redis cluster with 12k shards.

Redis doc also recommends using CLUSTER_SHARDS over CLUSTER_SLOTS

[optional Relevant Links:]

Any extra documentation required to understand the issue.

Guide contributeur

Direction de recherche
Examinez l'implémentation actuelle de la découverte de cluster Redis utilisant CLUSTER SLOTS dans envoy/source/common/redis/. Comprenez comment la topologie du cluster est récupérée et analysée. Implémentez ensuite la nouvelle API CLUSTER SHARDS en adaptant la logique d'analyse. Mettez à jour les tests dans envoy/test/common/redis/.
Stack technique
cppredis
Domaine
backendinfrastructure
Type d'issue
Performance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Redis
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65