Bug #38498
Task #38086: Enhance TYPO3 bootstrap mechanism
Autoloader performance improvement
Status: | Resolved | Start date: | 2012-06-30 | |
---|---|---|---|---|
Priority: | Should have | Due date: | ||
Assigned To: | Christian Kuhn | % Done: | 100% |
|
Category: | - | Spent time: | - | |
Target version: | 6.0.0-alpha3 | |||
TYPO3 Version: | 6.0 | Is Regression: | ||
PHP Version: | Sprint Focus: | |||
Complexity: |
Description
kcachegrind shows that 25% of all time rendering a fully cached page is lost in "loadCoreAndRegistry" --> strtolower is called 1090 times (intro package).
Associated revisions
[TASK] Autoloader performance improvement
If the autoload registry is read from cache, all class names are
already lowercased. The patch calls the lowercase logic only if
the registry files are not read from cache. The unit tests show
that nothing breaks with this patch.
Cachegrind shows ~22% of rendering time is spend in
loadCoreAndExtensionRegistry() on a fully cached page,
this is reduced to ~2% with the patch.
Change-Id: I07dc95b29bf05970061b5fb2f9a5d8a5ba194960
Resolves: #38498
Releases: 6.0
Reviewed-on: http://review.typo3.org/12494
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
[TASK] Autoloader performance improvement
If the autoload registry is read from cache, all class names are
already lowercased. The patch calls the lowercase logic only if
the registry files are not read from cache. The unit tests show
that nothing breaks with this patch.
Cachegrind shows ~22% of rendering time is spend in
loadCoreAndExtensionRegistry() on a fully cached page,
this is reduced to ~2% with the patch.
Change-Id: I07dc95b29bf05970061b5fb2f9a5d8a5ba194960
Resolves: #38498
Releases: 6.0
Reviewed-on: http://review.typo3.org/12494
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
History
#1 Updated by Christian Kuhn about 3 years ago
- Subject changed from Performance Improvement for Bootstrap to Autoloader performance improvement
#2 Updated by Gerrit Code Review about 3 years ago
- Status changed from New to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12494
#3 Updated by Christian Kuhn about 3 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset ae0790613ee5f43a9da3cb7438eead22b332e531.