Skip to content

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

Merged
merged 6 commits into from
Dec 18, 2016

Conversation

javiereguiluz
Copy link
Member

This fixes #6815.


; php.ini
realpath_cache_size=4096K
realpath_cache_ttl=600
Copy link
Member

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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cache

@xabbuh
Copy link
Member

xabbuh commented Sep 21, 2016

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

@javiereguiluz
Copy link
Member Author

@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).
Copy link
Member

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).

@xabbuh
Copy link
Member

xabbuh commented Dec 14, 2016

@javiereguiluz So should we now update here to reflect the deprecations made in symfony/symfony#20777?

@javiereguiluz
Copy link
Member Author

@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!

@weaverryan weaverryan merged commit e2abd22 into symfony:master Dec 18, 2016
weaverryan added a commit that referenced this pull request Dec 18, 2016
…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
weaverryan added a commit that referenced this pull request Dec 18, 2016
weaverryan added a commit that referenced this pull request Dec 18, 2016
weaverryan added a commit that referenced this pull request Dec 18, 2016
@weaverryan
Copy link
Member

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!

weaverryan added a commit that referenced this pull request Dec 18, 2016
* 2.7:
  [#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
  Fixed code example wrong prepend() method declaration
weaverryan added a commit that referenced this pull request Dec 18, 2016
* 2.8:
  [#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
  Fixed code example wrong prepend() method declaration
weaverryan added a commit that referenced this pull request Dec 18, 2016
* 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
weaverryan added a commit that referenced this pull request Dec 18, 2016
* 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
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.

Include a guide for speed up the initialization time.
5 participants