Task #50349
closed
Reduce SQL queries of page tree in workspaces
Added by Oliver Hader over 11 years ago. Updated about 7 years ago.
100%
Description
Nodes of the page tree are highlighted, if that particular node has versioned elements in the current workspace. However, this is determined by iterating over all tables and all records for each page. If having installed many extensions with many tables any many records, this will produce a lot of SQL queries.
The process of "does have versions" can be stopped once the first version for a page/node has been found.
Updated by Gerrit Code Review over 11 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 https://review.typo3.org/22528
Updated by Mathias Schreiber almost 10 years ago
- Target version set to 7.4 (Backend)
Hey Olly, wanna meet up to dig into ths one?
The idea itself is the right way.
Updated by Alexander Opitz over 9 years ago
Updated by Grigori Prokhorov over 9 years ago
Hi Alexander,
what speaks against including just https://review.typo3.org/#/c/38700/ in the next 6.2.x?
I'm asking since IF the situation occurs which this patch resolves, it actually makes the usage of the workspaces completely impossible.
The patch from #66231 as a "stand-alone" solution (as compared to https://review.typo3.org/#/c/22528/) only affects a very specific part of the core since the affected method is used in exactly those two classes which are included.
Best,
Grigori
Updated by Christian Opitz over 9 years ago
This issue makes the workspace feature nearly unusable. +1 for getting this into 6.2
Updated by Gerrit Code Review over 9 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/22528
Updated by Gerrit Code Review over 9 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/22528
Updated by Gerrit Code Review over 9 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/22528
Updated by Gerrit Code Review over 9 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/22528
Updated by Gerrit Code Review over 9 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/22528
Updated by Nicole Cordes over 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 37cd2d5084d48e77e17e8fb14ecbb5e55c7946c7.
Updated by Gerrit Code Review over 9 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/41425
Updated by Gerrit Code Review over 9 years ago
Patch set 2 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/41425
Updated by Nicole Cordes over 9 years ago
- Status changed from Under Review to Resolved
Applied in changeset d2c4bbbb34ffe31ad2007573158819b3ce4aee65.
Updated by Gerrit Code Review over 9 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/41553
Updated by Nicole Cordes over 9 years ago
- Status changed from Under Review to Resolved
Applied in changeset c45de676a41672f4780875e05fd389cbb66a8a96.
Updated by Gerrit Code Review over 9 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42541
Updated by Oliver Hader about 9 years ago
- Status changed from Under Review to Closed
Follow-up #69439
Updated by Oliver Hader about 9 years ago
- Status changed from Closed to Resolved
Updated by Riccardo De Contardi about 7 years ago
- Status changed from Resolved to Closed