Closed ajgajg1134 closed 1 week ago
Benchmark execution time: 2024-11-15 19:08:22
Comparing candidate commit 56caa3e6 in PR branch andrew.glaude/httpretry
with baseline commit 9ffc56b0 in branch main
.
Found 0 performance improvements and 0 performance regressions! Performance is the same for 51 metrics, 2 unstable metrics.
Omitted due to size.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 70.53%. Comparing base (
9ffc56b
) to head (56caa3e
).
What does this PR do?
Set the
Datadog-Send-Real-Http-Status
header to indicate to the agent that we are able to handle 429 responses and will appropriately back off and retry. Due to legacy compatibility reasons the default behavior of the trace agent for trace payloads is to always respond with a200
status code even if the trace-agent detects it is overwhelmed and will drop the payload. By setting this header the trace-agent will instead respond with a 429 giving us an opportunity to retry and decrease the likelihood of lost data.Motivation
We have retry logic but we aren't using it :(
Additional Notes
Anything else we should know when reviewing?
How to test the change?
This change is well covered by the unit tests