Task #82473

Make use of TYPO3s runtime caches instead of static in-method variables

Added by Stefan Froemken 10 months ago. Updated 6 months ago.

Status:
Resolved
Priority:
Should have
Assignee:
-
Category:
Backend API
Target version:
Start date:
2017-09-13
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
7.1
Tags:
Complexity:
Sprint Focus:

Description

Hello core Team,

in a CLI Script I have to process a really huge amount of records with many different page UIDs. Foreach PID a make a call to BackendUtility::getPagesTSconfig which has its own cache realized with static in-method variables, which grows and grows and exceeds my memory limitation.
It would be cool if you can switch these static variables into a runtine cache of TYPO3s CachingFramework as you have done in various other places within BackendUtility.

Stefan


Related issues

Related to TYPO3 Core - Bug #63629: High memory consumption in BackendUtility::getPagesTSconfig while using the DataHandler Resolved 2014-12-06

Associated revisions

Revision a2c71b79 (diff)
Added by Stefan Froemken 6 months ago

[TASK] Migrate getPagesTSconfig to runtime cache

BackendUtility::getPagesTSconfig() uses local
method static properties as cache.
This prevents dedicated flushing, which may
be needed in rare cases to prevent for instance
from exceeded memory during huge imports.
The patch switches that to cache framework
cache_runtime which is already used at other
places within BackendUtility.

Resolves: #82473
Related: #63629
Releases: master, 8.7
Change-Id: I4d54819f05a6c7b8ce175bc827b626277e2d4c04
Reviewed-on: https://review.typo3.org/54133
Tested-by: TYPO3com <>
Reviewed-by: Anja Leichsenring <>
Tested-by: Anja Leichsenring <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

Revision 6ba79f46 (diff)
Added by Stefan Froemken 6 months ago

[TASK] Migrate getPagesTSconfig to runtime cache

BackendUtility::getPagesTSconfig() uses local
method static properties as cache.
This prevents dedicated flushing, which may
be needed in rare cases to prevent for instance
from exceeded memory during huge imports.
The patch switches that to cache framework
cache_runtime which is already used at other
places within BackendUtility.

Resolves: #82473
Related: #63629
Releases: master, 8.7
Change-Id: I4d54819f05a6c7b8ce175bc827b626277e2d4c04
Reviewed-on: https://review.typo3.org/55451
Tested-by: TYPO3com <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

History

#1 Updated by Gerrit Code Review 10 months 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 https://review.typo3.org/54133

#2 Updated by Gerrit Code Review 10 months ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54133

#3 Updated by Gerrit Code Review 10 months ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54133

#4 Updated by Gerrit Code Review 9 months ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54133

#5 Updated by Gerrit Code Review 7 months ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54133

#6 Updated by Gerrit Code Review 7 months ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54133

#7 Updated by Gerrit Code Review 6 months ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54133

#8 Updated by Gerrit Code Review 6 months ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54133

#9 Updated by Christian Kuhn 6 months ago

  • Related to Bug #63629: High memory consumption in BackendUtility::getPagesTSconfig while using the DataHandler added

#10 Updated by Gerrit Code Review 6 months ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54133

#11 Updated by Gerrit Code Review 6 months ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/55451

#12 Updated by Stefan Froemken 6 months ago

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

Also available in: Atom PDF