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

Added by Steffen Ritter over 5 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
Start date:
2014-02-26
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Associated revisions

Revision 8830ba39 (diff)
Added by Benni Mack about 4 years ago

[!!!][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 <>
Tested-by: Nicole Cordes <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

History

#1 Updated by Christian Kuhn over 5 years 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 almost 5 years 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 Benni Mack over 4 years 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 over 4 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 http://review.typo3.org/40500

#5 Updated by Gerrit Code Review over 4 years 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 over 4 years 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 about 4 years 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 about 4 years 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 about 4 years 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 Benni Mack about 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#11 Updated by Riccardo De Contardi almost 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF