Skip to content

perf(discovery): improve discovery performance#2002

Merged
brendt merged 4 commits into
3.xfrom
discovery-performance
Feb 21, 2026
Merged

perf(discovery): improve discovery performance#2002
brendt merged 4 commits into
3.xfrom
discovery-performance

Conversation

@brendt

@brendt brendt commented Feb 20, 2026

Copy link
Copy Markdown
Member

No description provided.

@brendt brendt requested a review from innocenzi as a code owner February 20, 2026 14:11
@brendt brendt marked this pull request as draft February 20, 2026 14:11
@github-actions

github-actions Bot commented Feb 20, 2026

Copy link
Copy Markdown

Benchmark Results

Comparison of discovery-performance against 3.x (18e61b3b365ac724f38ef82ff213d3676757cd60).

Open to see the benchmark results
Benchmark Set Mem. Peak Time Variability
KernelBootBench(benchBoot) - 26.742mb +0.01% 101.648ms -17.15% ±0.51% -72.36%
ViewRenderBench(benchExpressions) - 23.903mb 0.00% 547.986μs +5.70% ±2.21% -14.36%
DiscoveryScanBench(benchFullDiscoveryScan) - 23.005mb +0.01% 51.185ms -27.85% ±0.71% -17.34%
DiscoveryScanBench(benchSinglePackageScan) console (large) 23.006mb +0.01% 6.400ms -24.44% ±0.42% -23.98%
DiscoveryScanBench(benchSinglePackageScan) router (large) 23.006mb +0.01% 7.479ms -13.02% ±1.02% +33.10%

Generated by phpbench against commit 7023043

@xHeaven

xHeaven commented Feb 20, 2026

Copy link
Copy Markdown
Member

Managed to shave off ~25-30% from discovery, see the updated benchmark above.

@brendt brendt marked this pull request as ready for review February 21, 2026 07:59
@brendt brendt requested a review from aidan-casey as a code owner February 21, 2026 07:59
@brendt

brendt commented Feb 21, 2026

Copy link
Copy Markdown
Member Author

Beautiful 😁

@brendt brendt merged commit 14add85 into 3.x Feb 21, 2026
82 checks passed
@brendt brendt deleted the discovery-performance branch February 21, 2026 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants