Uh oh!
There was an error while loading. Please reload this page.
test: stress bounded native pattern cache - #120
Conversation
There was a problem hiding this comment.
Pull request overview
Adds deterministic stress tests to validate that the instance-owned, bounded native pattern compiler cache remains capped at its configured capacity under high-cardinality sequential and concurrent admissions, while ensuring legacy RuntimeCompiler static caches are not mutated.
Changes:
- Introduces a sequential high-cardinality test proving cache plateauing and eviction via identity change on recompilation of the oldest key.
- Adds a concurrent admission stress test ensuring the cache remains bounded and legacy caches remain unchanged.
- Verifies evicted-but-still-referenced compiled patterns can still match without materializing the input (guarded
CharSequence).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This comment has been minimized.
This comment has been minimized.
31ae00b to
ab94994CompareCodecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@## agent/logs-reggie-l1j #120 +/- ##
=======================================================
Coverage 84.5% 84.5% Complexity 1 1 =======================================================
Files 156 156 Lines 43277 43277 Branches 5910 5910 =======================================================
Hits 36571 36571 Misses 4913 4913 Partials 1793 1793 Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
ab94994 to
7d6c90cCompare7d6c90c to
61206a8Compare
Summary
Adds deterministic high-cardinality stress coverage for the bounded native pattern compiler cache.
Validation
./gradlew :reggie-runtime:test --tests com.datadoghq.reggie.runtime.ReggieCompiledPatternCacheCardinalityTest./gradlew :reggie-runtime:test./gradlew spotlessApply