envoyproxy/envoy

CacheFilter: Handle backpressure from downstream

Open

#9.835 geöffnet am 25. Jan. 2020

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/cachehelp wanted

Repository-Metriken

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

Beschreibung

CacheFilter (#7198) doesn't yet handle backpressure from downstream, and could thus overflow downstream buffers when serving from cache. This must be addressed before CacheFilter will be suitable for production use.

Contributor Guide

Research-Richtung
Untersuchen Sie den CacheFilter Code unter source/extensions/filters/http/cache/cache filter.h und .cc. Verstehen Sie das aktuelle synchrone Caching Verhalten und wie es Backpressure ignoriert. Sehen Sie sich an, wie der Router Filter (source/extensions/filters/http/router/router.cc) mit der Flusskontrolle umgeht. Erwägen Sie die Implementierung von Watermark Callbacks oder die Verwendung des Envoy Puffers. Es gibt keine verknüpften PRs oder aktuelle Kommentare, beginnen Sie also mit dem Lesen der vorhandenen Implementierung und testen Sie mit einem Downstream, der Backpressure ausübt.
Tech Stack
cpp
Domain
backendinfrastructure
Issue Type
Fehler
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
C++Envoy proxy internalsBuffer management
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
25