envoyproxy/envoy

feature req: SQL codecs need to inject or parse trace headers from comments

Open

#11.718 geöffnet am 23. Juni 2020

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

Repository-Metriken

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

Beschreibung

Having the ability to trace database calls is very useful. Imagine the following example flow:

app---(sql query)-->sidecar--->DB

if app sends SQL queries with the trace headers as SQL comments, then the sidecar can emit client-side trace spans, recording the DB call durations. [The more industrious lot can parse DB query logs to extract trace headers and emit server side spans?]

if app doesn't send the trace headers as comments, the sidecar can still inject them (it will be a broken trace, but a trace nevertheless).

I am not sure if mongodb supports comments - the same can be done for mongo as well.

Contributor Guide

Research-Richtung
Untersuche, wie die SQL Codecs von Envoy SQL Abfragen parsen und Trace Header als Kommentare hinzufügen; schaue dir vorhandene Trace Implementierungen in Envoy an und wie Header in Abfragezeichenfolgen eingefügt werden.
Tech Stack
cpp
Domain
backendapinetworkingobservability
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.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
C++Envoytracing
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30