Problem description: Now that the import of large xlsx files has been fixed (see 30770), it is easier to get to work with really large spreadsheets. The sad fact is that they cannot be saved! Steps to reproduce: 1. Open http://www.elsevier.com/__data/assets/excel_doc/0003/148548/title_list.xlsx 2. Try saving as ods 3. See UI freezing 4. Wait 5. Note how cpu usage goes up to 100% 6. Wait 7. Wait 8. Listen how the fan of your laptop reaches full speed 9. Wait 10. Wait 11. Really, wait ... 1003. Give up and kill the process
On which env (Windows, Linux, MacOs) and version? What's the LO version?
I see this on linux, specifically Kubuntu 13.10 64 bit Libreoffice as distributed by libreoffice.org (not the ubuntu provided builds) On LibO 4.2.3 RC 2
In this specific file (title_list.xlsx) is a super large sheet1.xml which contains about 2.5 million xml elements (opening and closing). I will check how that is going to be converted when writing .ods files.
Tobias Lippert committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=798379313dca8de97e431ef2fe68129aaa1dcf04 fdo#76754 Speed up registration of new listeners to SfxBroadcaster The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
There are two more outstanding commits which have to be reviewed first. The time to save the provided file will be reduced to about 40s when they are integrated. I update the ticket when the changes are included.
Tobias Lippert committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=8f8ee6fd2c50572c44363a6d5d23bd9d192fe08d fdo#76754 Add Positions by StyleSheetFamily to IndexedStyleSheets The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Tobias Lippert committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=9782ba7f1b7f4d6cc11045b6c953deb6f17f321d fdo#76754 Add return first to IndexedStyleSheets to speed up ods writing The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Hello Sergio, the changes are available in the current master branch. You can check whether they work for you. (check the previous comment for instructions) I am working on speeding the saving up even further, but at least you will be able to save the file now.
I can save the file in the current master branch of LibreOffice in under a minute.
Tobias: do you plan some backport for 4.3 and perhaps 4.2 branches?
I did not have any plans to do it yet. But if you think it is an issue that is worth the backport, I can check whether it is feasible and do it. I work on LO in my free time, so it might take a month or so to do it. Is that ok?
Just saw the patches. I'm not an expert (I'd say rather the opposite :-)) but indeed, it could be time consuming to backport. I let you judge. Anyway, thank you for the quick feedback! :-)