Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
perf: refactor path logic in router + add LRU cache (vercel#41365)
I'm investigating the runtime perf of the node server and this was one of the hot spot I stumbled upon. This diff: - refactors getPagePath to not throw when it doesn't find a path: this function is used to check if a path exists, we don't want that kind of overhead there - adds a LRU cache to cache the result of the evaluation Results: before: > 110k requests in 11.01s, 285 MB read after >135k requests in 11.01s, 348 MB read on an autocannon run ## Bug - [ ] Related issues linked using `fixes #number` - [ ] Integration tests added - [ ] Errors have a helpful link attached, see `contributing.md` ## Feature - [ ] Implements an existing feature request or RFC. Make sure the feature request has been accepted for implementation before opening a PR. - [ ] Related issues linked using `fixes #number` - [ ] Integration tests added - [ ] Documentation added - [ ] Telemetry added. In case of a feature if it's used or not. - [ ] Errors have a helpful link attached, see `contributing.md` ## Documentation / Examples - [ ] Make sure the linting passes by running `pnpm lint` - [ ] The "examples guidelines" are followed from [our contributing doc](https://github.com/vercel/next.js/blob/canary/contributing/examples/adding-examples.md)
- Loading branch information