envoyproxy/envoy

Change Redis cluster discovery to use CLUSTER_SHARDS API

Open

#36.975 aberto em 4 de nov. de 2024

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
area/redisenhancementhelp wanted

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Examine a implementação atual da descoberta de cluster Redis usando CLUSTER SLOTS em envoy/source/common/redis/. Entenda como a topologia do cluster é buscada e analisada. Em seguida, implemente a nova API CLUSTER SHARDS, adaptando a lógica de parsing. Atualize os testes em envoy/test/common/redis/.
Pilha de tecnologia
cppredis
Domain
backendinfrastructure
Tipo Issue
Performance
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Redis
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65