fix(llmobs): flush span events buffer when it reaches certain size#4524
fix(llmobs): flush span events buffer when it reaches certain size#4524rarguelloF wants to merge 2 commits intomainfrom
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files
🚀 New features to boost your workflow:
|
|
✅ Tests 🎉 All green!❄️ No new flaky tests detected 🎯 Code Coverage (details) 🔗 Commit SHA: af49871 | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback! |
BenchmarksBenchmark execution time: 2026-04-13 15:19:32 Comparing candidate commit af49871 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 156 metrics, 8 unstable metrics.
|
| var lastVersion int | ||
| for i := 0; i < len(insert); i += chunkSize { | ||
| end := i + chunkSize | ||
| if end > len(insert) { |
There was a problem hiding this comment.
🚫 [golangci] reported by reviewdog 🐶
minmax: if statement can be modernized using min (modernize)
There was a problem hiding this comment.
🚫 [golangci] reported by reviewdog 🐶
rangeint: for loop can be modernized using range over int (modernize)
What does this PR do?
Motivation
Reviewer's Checklist
make lintlocally.make testlocally.make generatelocally.make fix-moduleslocally.Unsure? Have a question? Request a review!