Bug #38498

Task #38086: Enhance TYPO3 bootstrap mechanism

Autoloader performance improvement

Added by Susanne Moog about 3 years ago. Updated about 3 years ago.

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

Revision ae079061
Added by Christian Kuhn about 3 years ago

[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

Revision ccbd26bf
Added by Christian Kuhn about 3 years ago

[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

Also available in: Atom PDF