Task #90104

Introduce prepared statement for BackendUtility::getPageForRootline

Added by Markus Klein 12 days ago. Updated 11 days ago.

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

100%

TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

This function is called a lot, hence the DB statement should be reused.


Related issues

Related to TYPO3 Core - Epic #88474: Page tree performance in 9.5 New 2018-11-16

Associated revisions

Revision cd0c343a (diff)
Added by Markus Klein 11 days ago

[TASK] Use prepared statement in BackendUtility::getPageForRootline

The function is already caching the result for individual pages.
Still it is called for a lot of pages, so even more speed
can be gained by also re-using the database statement.

Specifically the page-tree rendering time reduces a lot with this.

Resolves: #90104
Releases: master, 9.5
Change-Id: I35308e163cd93b22c12ecf743759b2f19e2a7ad9
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/62867
Tested-by: TYPO3com <>
Tested-by: Benni Mack <>
Tested-by: Susanne Moog <>
Reviewed-by: Benni Mack <>
Reviewed-by: Susanne Moog <>

Revision a8116815 (diff)
Added by Markus Klein 11 days ago

[TASK] Use prepared statement in BackendUtility::getPageForRootline

The function is already caching the result for individual pages.
Still it is called for a lot of pages, so even more speed
can be gained by also re-using the database statement.

Specifically the page-tree rendering time reduces a lot with this.

Resolves: #90104
Releases: master, 9.5
Change-Id: I35308e163cd93b22c12ecf743759b2f19e2a7ad9
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/62902
Tested-by: TYPO3com <>
Tested-by: Susanne Moog <>
Reviewed-by: Susanne Moog <>

History

#1 Updated by Gerrit Code Review 12 days 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/c/Packages/TYPO3.CMS/+/62867

#2 Updated by Markus Klein 12 days ago

  • Related to Epic #88474: Page tree performance in 9.5 added

#3 Updated by Gerrit Code Review 11 days 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/c/Packages/TYPO3.CMS/+/62867

#4 Updated by Gerrit Code Review 11 days ago

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/c/Packages/TYPO3.CMS/+/62873

#5 Updated by Gerrit Code Review 11 days 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/c/Packages/TYPO3.CMS/+/62873

#6 Updated by Gerrit Code Review 11 days 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/c/Packages/TYPO3.CMS/+/62873

#7 Updated by Gerrit Code Review 11 days 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/c/Packages/TYPO3.CMS/+/62867

#8 Updated by Gerrit Code Review 11 days ago

Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/62902

#9 Updated by Markus Klein 11 days ago

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

Also available in: Atom PDF