Skip to content

Add route caching - #3876

Merged
josephsnyder merged 1 commit into
Kitware:masterfrom
williamjallen:route-cache
Jul 28, 2026
Merged

Add route caching#3876
josephsnyder merged 1 commit into
Kitware:masterfrom
williamjallen:route-cache

Conversation

@williamjallen

Copy link
Copy Markdown
Collaborator

Laravel recommends enabling route caching for production apps. Unfortunately, route caching does not work for apps hosted from a subdirectory, which led to it being removed in #2276. Local testing shows a roughly 40% improvement in baseline request handling time with route caching enabled. This PR re-adds route caching, but only for CDash instances hosted at the root path, allowing virtually all CDash instances to benefit from this performance improvement while continuing to support CDash-in-a-subdirectory instances.

Laravel recommends enabling route caching for production apps.  Unfortunately, route caching does not work for apps hosted from a subdirectory, which led to it being removed in Kitware#2276.  Local testing shows a roughly 40% improvement in baseline request handling time with route caching enabled.  This PR re-adds route caching, but only for CDash instances hosted at the root path, allowing virtually all CDash instances to benefit from this performance improvement while continuing to support CDash-in-a-subdirectory instances.
@josephsnyder
josephsnyder added this pull request to the merge queue Jul 27, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 27, 2026
@williamjallen
williamjallen added this pull request to the merge queue Jul 27, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 27, 2026
@josephsnyder
josephsnyder added this pull request to the merge queue Jul 28, 2026
Merged via the queue into Kitware:master with commit 1266823 Jul 28, 2026
7 checks passed
@williamjallen
williamjallen deleted the route-cache branch July 28, 2026 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants