Repository navigation
fix(test-optimization): use a 10-second flush interval - #10018
juan-fernandez merged 1 commit into
Conversation
Overall package sizeSelf size: 8.42 MB Dependency sizes| name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 3.3.3 | 125.43 kB | 445.14 kB | | opentracing | 0.14.7 | 194.81 kB | 194.81 kB | | dc-polyfill | 0.1.11 | 25.74 kB | 25.74 kB |🤖 This report was automatically generated by heaviest-objects-in-the-universe |
|
✅ All CI checks and tests passed. Datadog automation helped this PR pass. 🎉 All green!🧪 All tests passed 🔄 Datadog retried 1 test - 1 passed on retry 🎯 Code Coverage (details) 🔗 Commit SHA: dac7e36 | Docs | View more details | Give us feedback! |
There was a problem hiding this comment.
More details
The default flush interval changes to 10 seconds. The Jest worker interval stays at zero, and the test expectation matches the source value.
🤖 Datadog Autotest · Commit f2b1274 · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
BenchmarksBenchmark execution time: 2026-08-28 08:42:57 Comparing candidate commit dac7e36 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 2299 metrics, 11 unstable metrics.
|
f2b1274 to
f6a076a
Compare
|
@codex review |
|
Codex Review: Didn't find any major issues. Bravo. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
345b042 to
015f5c1
Compare
015f5c1 to
dac7e36
Compare
There was a problem hiding this comment.
Older Agent paths can receive test and suite events in one trace. The new export path drops the full trace and loses the test event.
🤖 Datadog Autotest · Commit 015f5c1 · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
|
@codex review |
|
/datadog autotest review |
|
Codex Review: Didn't find any major issues. Hooray! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
There was a problem hiding this comment.
More details
This PR changes the default Test Optimization flush interval from 5 seconds to 10 seconds.
Jest workers keep the zero interval.
The current PR diff contains only this change and its test update.
The current base fixes the earlier mixed-trace event loss.
No open findings remain.
🤖 Datadog Autotest · Commit dac7e36 · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
What does this PR do?
Changes the default Test Optimization periodic flush interval from 5 seconds to 10 seconds. Jest workers keep their existing zero interval.
Motivation
A longer periodic interval reduces small payload request frequency and gives the encoder more opportunity to batch events. The encoder size gate and final flush continue to send data independently of this interval.
This was extracted from #9987 so the two-line configuration change can be reviewed separately while #9987 remains the combined E2E validation branch. #10017 has merged, and this PR is now based directly on master.
Additional Notes
Verification: