envoyproxy/envoy

thread_local: thread safety follow ups

Open

#13.313 aperta il 29 set 2020

Vedi su GitHub
 (1 commento) (0 reazioni) (1 assegnatario)C++ (5373 fork)batch import
area/thread_localhelp wanted

Metriche repository

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

Descrizione

A couple of follow ups to https://github.com/envoyproxy/envoy/pull/12833:

  1. Try to make the API a bit cleaner with templating: https://github.com/envoyproxy/envoy/pull/12833#discussion_r496594468
  2. Build clang-tidy plugin to check for any TLS functions capturing "this", and fix all current examples. (There are several that capture "this" in the initial set() call, and some that have been "fixed" to use shared_from_this and weak_ptr.)

Guida contributor

Direzione di ricerca
Rivedi la PR collegata #12833 per il contesto sull'API thread local attuale e la discussione sui miglioramenti ai template. Indaga gli usi esistenti di thread local nella codebase di Envoy e identifica i punti in cui viene catturato 'this'. Per il plugin clang tidy, esamina i plugin clang tidy esistenti nel repository di Envoy o esempi esterni per comprendere l'integrazione di build. L'obiettivo è implementare i due task di follow up senza risolvere l'intero problema.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
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.
Chiara
Prerequisiti
C++clang tidy
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30