feat: [Trace Stats] Implement stats concentrator by lym953 · Pull Request #856 · DataDog/datadog-lambda-extension

@lym953 mentioned this pull request

Sep 19, 2025

lym953 added a commit that referenced this pull request

Sep 22, 2025
## This PR
1. Move stats generation after trace obfuscation, which is the correct
order as suggested by Trace Agent team. Right now stats generation is
before trace obfuscation.
2. Also generate trace stats for OTLP agent. Right now we only do it for
trace agent.

## Architecture
Copied from #842
<img width="1296" height="674" alt="image"
src="https://github.com/user-attachments/assets/2d4cb925-6cfc-4581-8ed6-6bd87cf0d87a"
/>

## Testing
Tested in the next PR
#856, which
implements stats concentrator. Trace stats appeared in Datadog.
<img width="538" height="317" alt="image"
src="https://github.com/user-attachments/assets/48b849cc-2413-41d5-8576-5ff657c21a0f"
/>


## Next steps
1. Implement `StatsConcentrator`
2. Rename for clarity:
  - `SendingTraceStatsProcessor` -> `TraceStatsGenerator`
  - `stats_sender` -> `stats_generator`
3. Small refactor: consider passing around `stats_sender` instead of
`stats_concentrator_handle`. Right now
`SendingTraceStatsProcessor::new()` is called in three places. It might
be possible to call it only once then pass it around.

## Notes
Jira: https://datadoghq.atlassian.net/browse/SVLS-7593

Base automatically changed from yiming.luo/trace-stats-6 to main

September 22, 2025 20:02

lym953

@lym953 lym953 marked this pull request as ready for review

September 23, 2025 18:49

litianningdatadog

duncanista

duncanista

duncanista

@lym953 lym953 deleted the yiming.luo/trace-stats-7 branch

September 25, 2025 19:31