Issue 101219

Summary: Use memory-cache for small streams in storage implementation.
Product: General Reporter: mikhail.voytenko
Component: codeAssignee: michael.ruess
Status: CLOSED FIXED QA Contact: issues@framework <issues>
Severity: Trivial    
Priority: P3 CC: issues, os_ooo, www.openoffice.org
Version: DEV300m45   
Target Milestone: OOo 3.2   
Hardware: All   
OS: All   
Issue Type: ENHANCEMENT Latest Confirmation in: ---
Developer Difficulty: ---
Attachments:
Description Flags
The performance measures. none

Description mikhail.voytenko 2009-04-20 11:56:18 UTC 
The transaction model of storages requires to cache data until the final commit
of the root storage. Using of the memory cache instead of temporary file cache
would improve the storage performance. Currently the maximal size of the memory
caching is limited, if the stream is bigger than the limit the usual temporary
file caching is used.
Comment 1 mikhail.voytenko 2009-04-20 14:10:03 UTC 
The change is commited in os128 cws.
As result, all the streams that are smaller than 30000 bytes are stored
temporally in the memory ( instead of temporary files ) during the
saving/loading process.
Comment 2 mikhail.voytenko 2009-04-20 14:13:07 UTC 
Created attachment 61689 [details]
The performance measures.
Comment 3 mikhail.voytenko 2009-04-20 14:17:54 UTC 
The attached performance measurements were done on DEV300_m45 with integrated
fix for this issue and original DEV300_m45 a the reference.
Comment 4 mikhail.voytenko 2009-04-21 08:20:43 UTC 
.
Comment 5 mikhail.voytenko 2009-05-15 10:49:34 UTC 
mav->mru: I see no way for QA to test directly the new implementation. What
still could be tested is that all the loading/storing automated tests ( for all
applications ) are still running successfully.
Comment 6 michael.ruess 2009-05-19 16:18:58 UTC 
Verified in CWS os128.
The automated and manual tests regarding load/save did not show any problems.
Comment 7 pmike 2009-06-11 22:00:49 UTC 
I've tested OOo-dev m50 with file from issue 70544.
Load perf is no longer suffer from many temp files, but OOo still creates/erases
them.
Comment 8 michael.ruess 2009-07-02 14:28:23 UTC 
Checked in DEV300m51.