envoyproxy/envoy

move redis to TCP connection pool

Open

#3.818 geöffnet am 9. Juli 2018

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/redishelp wantedtech debt

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

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:

Contributor Guide

Research-Richtung
Untersuchen Sie die Refaktorierung des HTTP1 Verbindungspools in PR #3747 und die Änderung des tcp proxy in #3938. Wenden Sie dann dasselbe Muster auf redis proxy an, indem Sie die direkte TCP Verbindungserstellung durch den generischen Verbindungspool ersetzen.
Tech Stack
cpp
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30