Task #54251

Epic #55070: Workpackages

Epic #55065: WP: Overall System Performance (Backend and Frontend)

Bug #52949: Speed decrease since 4.5

GeneralUtility::array_merge_recursive_overrule has bad performance

Added by Markus Klein over 8 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Performance
Target version:
Start date:
2013-12-06
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
5.5
Tags:
Complexity:
easy
Sprint Focus:

Description

According to my profiling activities, GeneralUtility::array_merge_recursive_overrule has really bad performance. (~ 111 ms own time for a BE call, that ranks number 3 in the list of the most expensive calls based on own time sorting.)
Some tests show that this is due to many array copy actions during the recursive calls (foreach constructs with $val being a copy of possibly large subarrays).

I propose to optimize this method by:
  1. making the first parameter by reference thus working on the original array
  2. move the function from GeneralUtility to ArrayUtility
  3. deprecate the function in GeneralUtility, but still work on a copy of the first array there to preserve backwards compatibility
  4. change all calls in core to use the new method

This brings the own time of the function down to 42 ms on my test system, getting the function out of the expensive calls in TYPO3 CMS.


Related issues

Related to TYPO3 Core - Task #57499: Improve deprecation message of GeneralUtitlity::array_merge_recursive_overrule()Closed2014-04-01

Actions
Related to TYPO3 Core - Bug #63812: Lower memory use in ArrayUtil->mergeRecursiveWithOverruleClosedStephan GroƟberndt2014-12-12

Actions

Also available in: Atom PDF