Skip to content

Commit 15e6933

Browse files
committed
Merge pull request #27 from Nyholm/patch-1
Added warning of enabling routing cache when you just have a few routes
2 parents fffe307 + 37cd239 commit 15e6933

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

README.md

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,9 +119,13 @@ cache:
119119
```
120120

121121
If you change any of your routes, you will need to clear the cache. If you use a cache implementation that supports
122-
tagging (implements [TaggablePoolTrait](https://github.com/php-cache/taggable-cache/blob/master/src/TaggablePoolInterface.php))
122+
tagging (implements [TaggablePoolInterface](https://github.com/php-cache/taggable-cache/blob/master/src/TaggablePoolInterface.php))
123123
you can clear the cache tagged with `routing`.
124124

125+
The routing cache will make the route lookup more performant when your application have many routes, especially many
126+
dynamic routes. If you just have a few routes your performance will actually be worse enabling this.
127+
Use [Blackfire](https://blackfire.io/) to profile your application to see if you should enable routing cache or not.
128+
125129

126130
### Clearing the cache
127131

0 commit comments

Comments
 (0)