Bug #17254
closed
Multiple retrieval of all table rows
Added by Miroslav Monkevic over 17 years ago. Updated about 11 years ago.
0%
Description
Here is my situation:
I have records containing "inline" field (A) which displays children containing inline element (B). Typical record contains A field with 10 children each of them having B element with 10 children displayed. It is only 100 simple records but rendering process takes up to 20 sec. Plus I had to uprise memory_limit up to 128Mb!
Looking onto the sources I noticed that when inline fields children are rendered, ALL of table's records are retrieved into memory and records titles are processed. This happens even if there no children to render.
So in my case, table A contains 68 rows, table B - 892.
If field B is empty - 68 records are retrieved and processed.
If field B has 10 children then 68 A records + 10 * 892 B records are retrieved and processed. If children's titles are rendered by user function then this function is called 9000 times. If it resides in non persistent user object then object is created 9000 times and so on...
Would it be possible to retrieve only records that are actually needed. I had no time to dig deeper into core on this matter. If there is greater reason for retrieving all record, maybe they could be at least cached to prevent multiple retrieval during one form rendering?
Thank you!
(issue imported from #M5511)
Updated by Miroslav Monkevic over 17 years ago
added 'foreign_table_where' => 'AND table.field = ###THIS_UID###' to inline field's TCA. Now first call of t3lib_BEfunc::exec_foreign_table_where_query in t3lib_transferData->selectAddForeign retrievs only related records.
No boost in speed nor memory usage.
Updated by Oliver Hader over 17 years ago
Thanks for the report. I'm going to have a look on this because also the next release, TYPO3 4.2, has the topic "performance".
Updated by Miroslav Monkevic over 17 years ago
Great news, Oliver! Thank you for your hard work!
I also think that issue 0005500 is very related to overall performance prob.
To my mind resolving it would give some significant boost to client side. In my case (3-level deep inline elements hierarchy with total amount of 100 records) client script reaches execution time limit and FF is continuously asking if I want stop/debug/continue script execution.
Updated by Miroslav Monkevic over 17 years ago
Hi again!
Some addition I forgot to mention(not very related to records retrieval).
There were a lot of IRRE fixes since this bug was opened, so maybe it is fixed already. Anyway I noticed while editing record, described above, that size of page is 17Mb.
Looking into sources I noticed that wizards contained by 3-rd level (grandchildren of main record) IRRE elements generate huge (around 10K) onFocus and onClick attributes.
Throwing out wizards, reduced page size to 4.5mb. Not sure if this is a bug though. :)
onfocus="TBE_EDITOR.palUrl('tx_vvlssfdb_cresult:211:1','starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group',549,'tx_vvlssfdb_cresult','211',1);" id="data_tx_vvlssfdb_cresult_211_hidden_0" /><input type="hidden" name="data[tx_vvlssfdb_cresult]211[hidden]" value="0" /><a href="#" onclick="this.blur();TBE_EDITOR.palUrl('tx_vvlssfdb_cresult:211:1','starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group,starttime,endtime,fe_group',549,'tx_vvlssfdb_cresult','211',0);return false;" title="tx_vvlssfdb_cresult">
Updated by Alexander Opitz over 11 years ago
- Status changed from Accepted to Needs Feedback
- Target version deleted (
0)
The issue is very old, does this issue exists in newer versions of TYPO3 CMS (4.5 or 6.1)?
Updated by Alexander Opitz about 11 years ago
- Status changed from Needs Feedback to Closed
- Assignee deleted (
Oliver Hader)
No feedback for over 90 days.