Project

General

Profile

Actions

Bug #17254

closed

Multiple retrieval of all table rows

Added by Miroslav Monkevic over 17 years ago. Updated about 11 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2007-04-27
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.1
PHP Version:
4.3
Tags:
Complexity:
Is Regression:
Sprint Focus:

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)


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Feature #17247: Load Inline Elements On DemandClosedSusanne Moog2007-04-26

Actions
Related to TYPO3 Core - Bug #18368: Many Relations will cause performance problemsClosed2008-03-04

Actions
Actions #1

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.

Actions #2

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".

Actions #3

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.

Actions #4

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">

Actions #5

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)?

Actions #6

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.

Actions

Also available in: Atom PDF