Story #81460
closed
Story #81432: Replace TYPO3 caching solution with PSR-6 / Doctrine capable alternative
Sacrifice caches' ill conceived getIdentifiersByTag
100%
Description
There are several reasons for needing to drop the getIdentifiersByTag method, but most important are:
- Tags are a concept designed for invalidation not retrieval
- Cache pools already accept "flushByTags" which serves invalidation purposes
- There is no way to reproduce this behavior via PSR-6 or PSR-16 interfaces
- Tags are handled differently by all engines and cannot be resolved externally without a large amount of processing (very likely including processing which must use Reflection to break the public contract)
- There is ONE single usage in all of TYPO3 which exists in the admin panel (a very legacy component) and is used to list the number of cached versions of a given page.
Sacrificing this single statistics output seems to be a reasonable choice - and will bring the TYPO3 concept of "cache tags" completely in line with the PSR-6 concept.
Updated by Gerrit Code Review almost 7 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53091
Updated by Gerrit Code Review almost 7 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53091
Updated by Gerrit Code Review almost 7 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53091
Updated by Gerrit Code Review almost 7 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53091
Updated by Gerrit Code Review almost 7 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53091
Updated by
Updated by Gerrit Code Review over 6 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53091
Updated by Gerrit Code Review over 6 years ago
Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53091
Updated by Gerrit Code Review over 6 years ago
Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53091
Updated by Gerrit Code Review over 6 years ago
Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53091
Updated by Gerrit Code Review over 6 years ago
Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53091
Updated by Anonymous over 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset badca6bea60329d80dc1a88eb57189402ba66951.
Updated by Georg Ringer over 5 years ago
- Related to Bug #34193: extGetNumberOfCachedPages provokes an allocate memory error added