Remove no-op segments from context when closed
Nessuno ha ancora preso questa issue.
- Lingua principale
- Java
- Stelle
- 100
- Fork
- 100
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
Backgroumd
When a customer calls beginSegmentWithSampling and the decision is made to not sample, a NoOpSegment is inserted into the context: https://github.com/aws/aws-xray-sdk-java/blob/d9d17ec980dce1c7e40a4b2e67cd5f76c5a36ea9/aws-xray-recorder-sdk-core/src/main/java/com/amazonaws/xray/AWSXRayRecorder.java#L431
However, when a customer calls close on that segment, it does not remove it from the context, leaving the stale no-op segment in context: https://github.com/aws/aws-xray-sdk-java/blob/5bab3e574f37b80b6b2628130eb7ea9b33336d4e/aws-xray-recorder-sdk-core/src/main/java/com/amazonaws/xray/entities/NoOpSegment.java#L391
This is distinct from SegmentImpl.close(), which does clear it from the context: https://github.com/aws/aws-xray-sdk-java/blob/5bab3e574f37b80b6b2628130eb7ea9b33336d4e/aws-xray-recorder-sdk-core/src/main/java/com/amazonaws/xray/entities/SegmentImpl.java#L200
Solution
We should consider removing the no-op segment from the context when closing it.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia in aws-xray-recorder-sdk-core/src/main/java/com/amazonaws/xray/entities/NoOpSegment.java, nell’implementazione di close, e confronta la relativa gestione del contesto con SegmentImpl.close(). Verifica come beginSegmentWithSampling inserisce il NoOpSegment in AWSXRayRecorder.java. Il lavoro è completato quando la chiusura di un segmento no-op lo rimuove dal contesto senza modificare il comportamento di SegmentImpl.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- java
- Ambito
- backend
- Tipo di issue
- Bug
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 55/100