Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing
|Summary:||[From Symphony]it's slow to open the simple xslx file|
|Component:||open-import||Assignee:||AOO issues mailing list <issues>|
|Status:||CONFIRMED ---||QA Contact:|
|Issue Type:||DEFECT||Latest Confirmation in:||---|
Description bjcheny 2012-08-20 02:27:41 UTC
Created attachment 79003 [details] Sample xslx takes long time to open It takes long time to open simple xslx file.
Comment 1 bjcheny 2012-08-20 03:10:21 UTC
It takes almost 300 sec to open sample file on my pc. I am working on patch for this problem.
Comment 2 bjcheny 2012-08-24 09:32:05 UTC
Created attachment 79126 [details] patch for performance of importing cell style in xslx The cause is StyleSheetChanged/PageStyleModified action much too frequently. Thus the patch disables it, and will only do StyleSheetChanged/PageStyleModified once after all properties are set.The changes are: 1. Split the SetOnePropertyValue into SetOnePropertyValueWithoutUpdate and StyleSheetChanged, 2. Only invoke StyleSheetChanged once after all properties are set by SetOnePropertyValueWithoutUpdate . It can improve the performance of importing cell style significantly.
Comment 3 bjcheny 2012-08-27 06:43:03 UTC
Created attachment 79163 [details] patch for performance of importing cell style in xslx - part 2 Another patch for performance improvement for importing cell styles in xslx. In this sample, there are up to 28380 records of user-defined Xf to import. It consumes a lot of time. In fact, only 4050 Xf records can be supported in sc module. Thus the patch modifies total count of Xf to import to 4050 in oox module,if exceeding.
Comment 4 Terry Yang 2012-08-29 09:46:52 UTC
Confirm this bug at my side, change bug to confirm status
Comment 5 Yolanda Zhang Ying 2012-08-30 02:09:36 UTC
Some review comments: For part1: I think it's ok to do so to reduce import time by limit the useless operation. For part2: I think it does not make sense to do so, how could we get the 4050 for max support No.? I don't think for OOXML file import, it has such limit for AOO to support more than 4050 styles. Pls help confirm this limitation, thx!