Skip to content
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

Added proper opcache configuration to improve performance #945

Open
wants to merge 1 commit into
base: 2.0
Choose a base branch
from

Conversation

oliverde8
Copy link
Contributor

@oliverde8 oliverde8 commented May 18, 2018

If we leave the default configuration opcache is not utilized at it's best.

Calling opcache_get_status(false); will return a cache_full. This means caches are invalidated quite often.

The issue comes from 2 points :

  • Akeneo actually needs nearly 200mb of space. so the default 64 is not enough.
  • There is around 15K php file in an Akeneo project so we need to increase the number of files as well to be on the safe side.

This has a non-negligible impact on performances on my dev machine, I go from ~600ms to ~500ms for loading a product page.

Let's note that if multiple php projects run on the same server these values need to take the sum of the needs of all projects.

This should also be changed in the docker files.

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