Issue 101219 - Use memory-cache for small streams in storage implementation.
Summary: Use memory-cache for small streams in storage implementation.
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: code (show other issues)
Version: DEV300m45
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo 3.2
Assignee: michael.ruess
QA Contact: issues@framework
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-04-20 11:56 UTC by mikhail.voytenko
Modified: 2009-07-02 14:28 UTC (History)
3 users (show)

See Also:
Issue Type: ENHANCEMENT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
The performance measures. (5.89 KB, text/html)
2009-04-20 14:13 UTC, mikhail.voytenko 		no flags Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
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.