envoyproxy/envoy

adaptive concurrency: Allow specification of runtime names

Open

#8638 aperta il 17 ott 2019

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

As a prerequisite to supporting both egress and ingress adaptive concurrency, it will be necessary to specify the runtime configuration names to avoid conflicts. Before declaring the filter production ready, the adaptive concurrency configuration parameters should be replaced by their runtime counterparts.

To be more specific for anyone who gets to this before I do, we want to change any parameters in the adaptive_concurrency.proto that correspond to any of the runtime options that override them. For example, changing parameters in the proto from Uint32 to RuntimeUInt32.

Guida contributor

Direzione di ricerca
Studia il file adaptive concurrency.proto e le corrispondenti opzioni runtime. Identifica i parametri che hanno override runtime (es. minRtt, sampleAggregatePercentile) e cambia il loro tipo da UInt32 a RuntimeUInt32. Aggiorna l'implementazione del filtro per leggere dai valori runtime quando disponibili.
Tech stack
cpp
Dominio
backendnetworking
Tipo issue
Funzionalità
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.
Abbastanza chiara
Prerequisiti
C++ProtobufEnvoy internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30