envoyproxy/envoy

move redis to TCP connection pool

Aperta

#3818 aperta il 9 lug 2018

Vedi su GitHubGuida contributor
 (6 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/redishelp wantedtech debt

Metriche repository

Star
 (27.997 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Tracking ticket for follow-on modifications related to the generic TCP connection pool, per discussion in #3747:

  • refactor HTTP1 connection pool to extend it
  • modify tcp_proxy to use connection pool vs. tcpConnForCluster
  • modify redis_proxy to use connection pool

Other possible lines of inquiry:

Guida contributor

Direzione di ricerca
Esamina il refactoring del pool di connessioni HTTP1 nella PR #3747 e la modifica di tcp proxy nella #3938. Quindi, applica lo stesso pattern a redis proxy sostituendo la creazione diretta di connessioni TCP con il pool di connessioni generico.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30