Skip to content

Q: APCu #775

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

Closed
kironet opened this issue Mar 23, 2018 · 4 comments · Fixed by #791
Closed

Q: APCu #775

kironet opened this issue Mar 23, 2018 · 4 comments · Fixed by #791
Labels

Comments

@kironet
Copy link

kironet commented Mar 23, 2018

Hi,

Why you have symfony/polyfill-apcu installed, but you never use APCu in config/controller?

Is it still recommended to use APC(u) with Symfony?

Thanks

@stof
Copy link
Member

stof commented Mar 23, 2018

@javiereguiluz does the currently version of the demo still relies on APCu as its caching layer or no ?

@stof
Copy link
Member

stof commented Mar 23, 2018

hmm, in any case, the APCu polyfill is useless. It is here to emulate APCu on top of APC for older PHP versions, but the demo requires PHP 7.1+, so you either have APCu or nothing at all (and then the polyfill cannot help much)

@javiereguiluz
Copy link
Member

We have this file: https://github.com/symfony/demo/blob/master/config/packages/prod/framework.yaml

#framework:
#    cache:
#        system: cache.adapter.apcu

@stof
Copy link
Member

stof commented Mar 23, 2018

but anyway, we don't really need the APCu polyfill, due to our PHP version requirement

javiereguiluz added a commit that referenced this issue Apr 25, 2018
This PR was merged into the master branch.

Discussion
----------

Removed symfony/polyfill-apcu dependency

This fixes #775.

Commits
-------

287abd6 Removed symfony/polyfill-apcu dependency
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants