envoyproxy/envoy

adaptive concurrency: Allow specification of runtime names

Open

#8 638 ouverte le 17 oct. 2019

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Étudiez le fichier adaptive concurrency.proto et les options d'exécution correspondantes. Identifiez les paramètres qui ont des remplacements d'exécution (par exemple, minRtt, sampleAggregatePercentile) et modifiez leurs types de UInt32 en RuntimeUInt32. Mettez à jour l'implémentation du filtre pour lire les valeurs d'exécution lorsqu'elles sont disponibles.
Stack technique
cpp
Domaine
backendnetworking
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
C++ProtobufEnvoy internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30