Apache OpenOffice (AOO) Bugzilla – Issue 121200
[Regression][Automation][Performance]Severe downgrade to save xls sample files.
Last modified: 2014-02-27 21:58:32 UTC
Build: rev. 1391367 OS: Windows Reproduce steps: 1. Launch AOO, opened sample file 2. Save it as original file type, recorded the saving time. Sample file: https://svn.apache.org/repos/asf/incubator/ooo/trunk/test/testgui/data/pvt/plain_11s.xls https://svn.apache.org/repos/asf/incubator/ooo/trunk/test/testgui/data/pvt/complex_29s.xls Manual testing of xls saving time between rev. 1390913 and rev. 1391367. The downgrade from regression is obvious huge: rev. 1390913 VS rev. 1391367 plain_11s.xls: 1.182 vs 18.226 complex_29s.xls: 4.448 vs 35.438
The performance regression was probably introduced by revision 1391359 #i121136 Improve Spreadsheet performance in some area (Patch 1) but it can no longer be reproduced on a recent trunk version probably because revision 1395911 fixed it and enabled other optimizations: #i121136 Improve Spreadsheet performance in some area (Patch 2)
I checked build r1401602 manually, it still has this problem.
It may be related with bug i119707, we will continue to do investigation
Roll back part codes of solutions for i119707. Submit patch in revision 1406980
r1407366 still have downgrade problem. Pls see http://people.apache.org/~liuzhe/testdashboard/#pvt_gui_Benchmark : build 350m1(Build:9611)-2012-11-10.
According to PVT result, the downgrade issue is not fixed, so reopen it
I can't reproduce the problem for plain_11s.xls but I don't have measure the saving time for the older revision. Saving complex_29s.xls seems to be slow and towards the end of the saving process I get a warning box that says The document contains more rows than supported in the selected format. Additional rows were not saved.
(In reply to hdu@apache.org from comment #1) > The performance regression was probably introduced by revision 1391359 > #i121136 Improve Spreadsheet performance in some area (Patch 1) > but it can no longer be reproduced on a recent trunk version probably > because revision 1395911 fixed it and enabled other optimizations: > #i121136 Improve Spreadsheet performance in some area (Patch 2) Can you provide any details why these changes could have a negative performance impact? Bug 121136 does not contain this information.
Not sure that I can still reproduce slow loading of complex_29.xls. The first two steps of the loading (loading and updating row heights) is quite fast (a couple of seconds). Only the third part (calculation of cell contents) takes a little longer (maybe 10s).
Save times are machine dependent, but both saves are close to left column of the table in original Description with AOO410 branch version as pro build. Definitely not close to the 2nd column.