[TYPO3-core] RFC: #8515: Speed up tslib_cObj->getTreeList by caching its results

[Dmitry Dulepov [typo3]]

Hi!

Ingo Renner wrote:
> In fact, at least during my tests there were only a few entries in the 
> table, but those helped to speed up the system quite a lot -  so can 
> rest calm =)

With small number of entries - yes. Index will be used for them and there will be no many seeks for data. But what about real life data?

> Sure, but as said it will benefit not only those cases, but _all_ 
> extensions which properly use the pibase API.

Correction: only those that use getTreeList ;)

> Performance is always needed, isn't it? ;) Other than that, hard disk 
> space is cheap nowadays.

It is not about hard disk space :) It is about how MySQl works. If you get 100000 records or so in your cache table, it will be slow on SELECT because indexes will not be used and full table scan will be done. It will end up in many second delays while searching for cached data. May be it is faster to perform recursive scans in this case.

I want to clarify once again: I am not against the patch, I only have small concerns that it may actually slow down the server under some conditions.

-- 
Dmitry Dulepov
TYPO3 Core team
More about TYPO3: http://typo3bloke.net/
Subscribe: http://typo3bloke.net/rss.xml