search

ONLYOFFICE / DesktopEditors

An office suite that combines text, spreadsheet and presentation editors allowing to create, view and edit local documents
https://www.onlyoffice.com
Other
2.76k stars 318 forks source link

Onlyoffice Spreadsheet can't import CSVs with a few megabytes #1297

Open Lancer11211 opened 1 year ago

Lancer11211 commented 1 year ago

This issue is unique.

Operating System

Linux (DEB package)

Version information

7.3.0.184

Expected Behavior

Onlyoffice should import the CSV.

Actual Behavior

Runs out of memory in less than a minute. The ones I've tried were just 5-6MBs and onlyoffice couldn't handle them.

Reproduction Steps

  1. Import a CSV with a size of ~5MB
  2. Wait
  3. See the crash

Additional information

No response

HectorB-2020 commented 1 year ago

@Lancer11211, how many lines does the file have? Is is a size of the file or a number of lines within? Somebody reported about an issue related to number of lines: https://github.com/ONLYOFFICE/DocumentServer/issues/2248

Also for the sake of completeness please see other issues about importing CSV:

Lancer11211 commented 1 year ago

It has 12K lines, each line is around 300 characters long. The file's size is 5.3Mb, it's mostly float numbers. Libreoffice Calc imported it in a few seconds. I've retried it again today but no luck.

Lancer11211 commented 1 year ago

The latest version - 7.3.3.50 - is much more efficient now and can load CSVs up to 33Mb. With anything larger, it shows an error popup: "An error happened". With the 33Mb file, it uses 857Mb RAM the first second, then drops to 400Mb in a few seconds - this means the GC is doing its job. Maybe it can't load bigger files due to some memory limit.

rsnegirev commented 1 year ago

Hi @Lancer11211

This is known issue - 51939 in our internal issue tracker. Thank you for report.

HectorB-2020 commented 1 year ago

@rsnegirev, if this is a know issue, will you please share the limitation which applies to imported CSV files?

rsnegirev commented 1 year ago

@HectorB-2020 we will add this information to the documentation and let you know.

https://helpcenter.onlyoffice.com/onlyoffice-editors/onlyoffice-spreadsheet-editor/HelpfulHints/ImportData.aspx