Bug #56319
Epic #55070: Workpackages
Epic #55065: WP: Overall System Performance (Backend and Frontend)
Bug #52949: Speed decrease since 4.5
Bug #56307: Findings from Bootstrap/PackageManager/ClassLoader analyses
Merge Cache/Cache and CacheFactory into CacheManager
| Status: | Resolved | Start date: | 2014-02-26 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | - | % Done: | 100% |
|
| Category: | - | Spent time: | - | |
| Target version: | 7.4 (Backend) | |||
| TYPO3 Version: | 7 | Is Regression: | No | |
| PHP Version: | Sprint Focus: | |||
| Complexity: |
Associated revisions
[!!!][TASK] Remove Cache helper class
The Cache class that acted as a central point when the caching
framework was optionally enabled for 4.3 is now moved
into separate methods.
The duplicate code for creating the database statement is now
moved from the Install Tool to EXT:core to be used in the Extension
Manager as well.
Resolves: #56319
Releases: master
Change-Id: I871a9a6ea506795335e2e8f5b6a410604feba506
Reviewed-on: http://review.typo3.org/40500
Reviewed-by: Nicole Cordes <typo3@cordes.co>
Tested-by: Nicole Cordes <typo3@cordes.co>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
History
#1 Updated by Christian Kuhn over 1 year ago
-1, imho we should keep this separation as long as flow does not merge cacheManager and cacheFactory into one. also, cache is a "connection" class between cms and flow code, so we should keep it for separation, too.
#2 Updated by Christian Kuhn 8 months ago
- Status changed from New to Rejected
Imho, this area is not a real bottleneck and it does not make much sense to sacrifice architecture in favor of (negligible) more performance.
For now, I'll close this issue as rejected.
#3 Updated by Benjamin Mack about 1 month ago
- Status changed from Rejected to New
- Target version set to 7.4 (Backend)
- TYPO3 Version changed from 6.2 to 7
We should get this fixed in CMS7.
#4 Updated by Gerrit Code Review about 1 month 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 http://review.typo3.org/40500
#5 Updated by Gerrit Code Review 21 days ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40500
#6 Updated by Gerrit Code Review 19 days ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40500
#7 Updated by Gerrit Code Review 19 days ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40500
#8 Updated by Gerrit Code Review 19 days ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40500
#9 Updated by Gerrit Code Review 19 days ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40500
#10 Updated by Benjamin Mack 18 days ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 8830ba39261a2d091871640dc561b7844bd8a747.