Apache OpenOffice (AOO) Bugzilla – Issue 52328
Performance: configmanager
Last modified: 2011-01-21 17:40:03 UTC
Some possible space for improvements: - reduceExpandedForLocale: modify localized props in place instead of copying entire structure - LocalStratumBase::isMoreRecent() optimize for empty query timestamp - TreeImpl::getDataAccessor - reuse same accessor, if we are in the same segment - CacheController::addComponentData - translation from INode-data to flat data is expensive
Handing over @sb: In this issue I noted some more or less low-hanging fruit for optimizing performance, which I found by looking at MT's profiling data. It is more a tracking issue. An individual issue should be filed for each individual problem as it is addressed. Of the listed items, the last one is a rather large change and a serious amount of work. The isMoreRecent change can avoid a lot of unnecessary file stats (but these files are stated later during the cache check, so the gain is not as big as one could hope). The other changes eliminate unneeded (probably) allocations (reduce...Locale) and mutexes (the allocator one). For more details get back to me.
.
The 'Accessor' issue is nailed in CWS configrefactor01 - since we remove 'Accessor' altogether :-) It's interesting though, that my profiles (admittedly on the re-factored code), don't show any of these areas as significant - but perhaps I'm missing something (?) Also - Mutex wise, I've CWS configrefactor01 removes ~all locking in favour of one big lock, which substantially simplifies the code (& accelerates it).
meanwhile, configmgr code has been re-written completely