[FrameworkBundle] Add APCu support for the Serializer #17025
Conversation
|
Why is this service named serializer.mapping.cache.apcu instead of serializer.mapping.cache.doctrine.apcu like #16822 ? |
|
@layanto : IMO the But we won't have any issue to name the new APCu implementation |
|
I too do not like overly long name but I think it is confusing that validator.mapping.cache.apc is deprecated in favour of validator.mapping.cache.doctrine.apc and then this adds validator.mapping.cache.apcu which is very similar to the deprecated name. |
|
@layanto APCu migration is a chance to have coherent and short names again. |
|
Closing this one as we should use the new caching component now. |
This PR configures support for the new APCu class from Doctrine Cache (doctrine/cache#115) for the Serializer service.
I'll open a similar PR for the validator service when #16822 will be merged.
I've opened it against master as a new feature, but maybe must it be merged in older versions (2.3?) and considered as a bug fix: APCu for PHP7 doesn't support the APC API anymore. So Symfony projects migrating to PHP7 will not be able to use metadata cache (mandatory in production) without APCu support or adding manually some services.