-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Updated the article about performance tuning tips #6937
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
||
; php.ini | ||
realpath_cache_size=4096K | ||
realpath_cache_ttl=600 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need to indent with 4 spaces
@@ -43,6 +43,25 @@ your ``php.ini`` configuration. | |||
.. index:: | |||
single: Performance; Autoloader | |||
|
|||
Configure the PHP realpath cache |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cache
I think we should add a tip that you need to clear the Opcache when you deploy application (we do something similar in the Composer autoloader section). Status: Needs work |
@xabbuh I made the requested changes. |
your ``php.ini`` configuration. | ||
For the same reasons, the byte code cache must also be cleared when deploying | ||
the application (for example by calling ``apc_clear_cache()`` PHP function when | ||
using APC and ``opcache_reset()`` when using OPCache). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should also note that the CLI and the web process do not share the same Opache (which will otherwise probably trick you when you clear the Opcache through your deploy script only for the CLI environment).
@javiereguiluz So should we now update here to reflect the deprecations made in symfony/symfony#20777? |
@xabbuh to be honest I'd like this to be merged .... because I want to make a lot of changes in it. I'll add the apcu thing to the list of future changes. Thanks! |
…ereguiluz) This PR was merged into the master branch. Discussion ---------- Updated the article about performance tuning tips This fixes #6815. Commits ------- e2abd22 Added a note about the different caches for CLI and the server b9d94b8 Added a mention to opcache.max_accelerated_files c1b64cc Further rewordings and improvements fab7a0a Minor reword 6409fc5 Fixes and improvements dd057dd Updated the article about performance tuning tips
Merged! With a few small tweaks. I accidentally did not switch the branch, so I ported it back to 2.7 at sha: 9cb25d9. Thanks! |
* 3.1: [#6937] A few small tweaks! Porting #6937 back to 2.7 Remove yourself word in acl doc page %kernel.root_dir%/app/data/data.sqlite -> %kernel.root_dir%/data/data.sqlite Fix sample sqlite database path in doctrine config Fix sample sqlite database path in doctrine config Fixed code example wrong prepend() method declaration
* 3.2: [#6937] A few small tweaks! Porting #6937 back to 2.7 Remove yourself word in acl doc page %kernel.root_dir%/app/data/data.sqlite -> %kernel.root_dir%/data/data.sqlite Fix sample sqlite database path in doctrine config Fix sample sqlite database path in doctrine config Fixed code example wrong prepend() method declaration
This fixes #6815.