perf(graph): do not rebuild TestFileFilter when the session already has one - #836
Performance Regression: -49.02%
⚠️ Different runtime environments detected
Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.
❌ 2 regressed benchmarks
✅ 82 untouched benchmarks
🆕 4 new benchmarks
⏩ 4 skipped benchmarks1
Warning
Please fix the performance issues or acknowledge them on CodSpeed.
Performance Changes
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Memory | extract |
213.6 KB | 518.7 KB | -58.82% |
| ❌ | Simulation | extract |
10.4 ms | 16.4 ms | -36.89% |
| 🆕 | Simulation | graph_gates_build_session[1] |
N/A | 124.4 ms | N/A |
| 🆕 | Simulation | graph_gates_build_session[4] |
N/A | 126.1 ms | N/A |
| 🆕 | Memory | graph_gates_build_session[1] |
N/A | 4.5 MB | N/A |
| 🆕 | Memory | graph_gates_build_session[4] |
N/A | 5.7 MB | N/A |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing execute-plan/79dd0742-pr-6-reuse-prepared-testfilefilter-during-graph-config (5cfe6cd) with execute-plan/79dd0742-pr-5-normalized-path-membership-without-pathcmp (b887520)
Footnotes
-
4 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩