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

Remove CacheFile cache from APC on write #6315

Merged
merged 1 commit into from Sep 28, 2014
Merged

Remove CacheFile cache from APC on write #6315

merged 1 commit into from Sep 28, 2014

Conversation

brownoxford
Copy link
Contributor

If APC is enabled on the piwik server, cache files written by the CacheFile
class are stored in the opcode cache. This causes disk io problems when doing
things like loading weblogs into piwik with the import_logs.py script,
especially when running in --disable-bulk-tracking mode. In this scenario,
apache write IO quickly reaches 100% and slows the import to a crawl.

If APC is enabled on the piwik server, cache files written by the CacheFile
class are stored in the opcode cache. This causes disk io problems when doing
things like loading weblogs into piwik with the import_logs.py script,
especially when running in --disable-bulk-tracking mode. In this scenario,
apache write IO quickly reaches 100% and slows the import to a crawl.
@brownoxford
Copy link
Contributor Author

Hmm. Seems like all of the tests that failed were related to composer install failing?

mattab pushed a commit that referenced this pull request Sep 28, 2014
…n-write

Remove CacheFile cache from APC on write
@mattab mattab merged commit 76865be into matomo-org:master Sep 28, 2014
@mattab
Copy link
Member

mattab commented Sep 28, 2014

Sorry about that! This is an issue when new contributors like you are so kindly doing a PR but our build was already failing unfortunately... we really want to avoid this in the future see #5954

PR looks good, this is a nice edge case improvement!

@mattab mattab added the Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change. label Sep 28, 2014
@brownoxford brownoxford deleted the delete-cachefile-from-apc-on-write branch September 30, 2014 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Task Indicates an issue is neither a feature nor a bug and it's purely a "technical" change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants