envoyproxy/envoy

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

Open

#11.718 aperta il 23 giu 2020

Vedi su GitHub
 (1 commento) (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

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.

Guida contributor

Direzione di ricerca
Indaga come i codec SQL di Envoy analizzano le query SQL e aggiungono le intestazioni di traccia come commenti; esamina le implementazioni di tracciamento esistenti in Envoy e come iniettare le intestazioni nelle stringhe di query.
Tech stack
cpp
Dominio
backendapinetworkingobservability
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
C++Envoytracing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30