envoyproxy/envoy

stats: allow pre-defined tags

Open

#14.071 geöffnet am 17. Nov. 2020

Auf GitHub ansehen
 (7 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/statshelp wanted

Repository-Metriken

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

Beschreibung

The internal API to create tags has two versions:

  1. Create from qualified stat name.
  2. Create from stat name plus a set of tags.

We'd like to have a third version:

  1. Create from stat name plus a set of tag names. Tag names match against the content of the stat to obtain values. For example, stat "a.b.c" with tag names "d.e" produces "a{d=b,e=c}". Here the separator is a dot, but could be any special character.

The benefit is two-fold:

  1. Save on space, instead of "a.d.b.e.c" we can share "d" and "e" symbols between stat names with different tag values.
  2. Simplify user experience by not required regexes to achieve the same effect.

Contributor Guide

Research-Richtung
Untersuchen Sie die aktuelle Tag Erstellungs API in source/common/stats/ und implementieren Sie eine neue Funktion, die einen Statistiknamen und eine Liste von Tag Namen akzeptiert und einen Punkt (oder ein anderes Trennzeichen) verwendet, um Werte aus dem Statistiknamen zu extrahieren.
Tech Stack
cpp
Domain
backendobservability
Issue Type
Funktion
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.
Halber Tag
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.
65