Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v5.2 High CPU/RAM when saving a large library and field formatters (normalize date) are enabled #7265

Open
1 task
alfureu opened this issue Dec 28, 2020 · 10 comments

Comments

@alfureu
Copy link

alfureu commented Dec 28, 2020

JabRef 5.2--2020-12-24--6a2a512
Windows 10 10.0 amd64
Java 14.0.2

This issue is present on the v5.2 stable.

Steps to reproduce the behavior:

  1. Create a larger file with 3000+ entries
  2. Try to save it...

Cannot help myself but it is really disappointing to see JabRef v5.2 still hogged when saving a large(r) library. I have 3000 entries, and whenever I hit Ctrl+S the computer fans start to ramp up and the status of JabRef in Task Manager goes to "Not responding"... This goes on for minutes (approx 5) until things settle (who know what is a problem with saving a txt file)!

image

This is really upsetting with a stable version, as it makes serious work almost impossible. I am sorry but default CPU usage around 20% on an 8-Core i7 7700HQ - 32GB RAM - 1TB NVMe SSD is just simply unacceptable.

@alfureu
Copy link
Author

alfureu commented Dec 28, 2020

Update: in case anybody wants to hunt some bugs, I disabled in the Library properties the Enable field formatters and things got a bit better. I believe we all acknowledge that this is sub-optimal.

@Ali96kz
Copy link
Contributor

Ali96kz commented Mar 11, 2021

@DOFfactory could you provide your file?

@alfureu
Copy link
Author

alfureu commented Mar 11, 2021

I do not have it with me anymore. However, it happens with any file, I had multiple field formatters just to make every entry properly unified, which I disabled and since then things go smooth.

@alfureu
Copy link
Author

alfureu commented Mar 12, 2021

image

@Ali96kz
Copy link
Contributor

Ali96kz commented Mar 12, 2021

I am working on it, I found a problem

@Ali96kz
Copy link
Contributor

Ali96kz commented Mar 15, 2021

Problems seems to be in org.jabref.model.entry.Date#parse(java.lang.String)
It may throw a lot of exception, but I can't reproduce it with my files.
If anyone will have the same problem, please attach your .bibfile

@alfureu
Copy link
Author

alfureu commented Mar 18, 2021

I think you might be right, as I usually scrape large amounts of literature from multiple databases (Scopus, SpringerLink, WOS, etc.) with Zotero's browser addon, then I export them into the corresponding bib file(s). Now, what I noticed that various databases provide a variety of date formats, e.g. 2010, 2010-05, 2010-May, some provide separate year + month fields.

I think it has been highlighted earlier in some other bugreports that there should be somehow an automatic process of unifying the essential/required fields, which is partially provided by the field formatters (but not ideally). IMHO the date field should be one of these, meaning an irregular date field should either use the year field if it has only 2010, a year and month fields if it has 2010-05 (while day is missing) in it. Even an automated 2010-00-00 would be better than what we have now, no?

Personally I would prefer using only the year field in the main required fields section (using biblatex format), but I guess the decision has been made somewhere along the line for the date field, which is now causing issues with the overall stability of JabRef if used with field formatters.

@Ali96kz Ali96kz mentioned this issue Mar 23, 2021
5 tasks
@calixtus
Copy link
Member

Hi @DOFfactory , we just merged a PR that should hopefully speed up parsing of Dates. Would you be so kind to test the current main-branch-build (we just renamed the master-branch), if that helped to relax the problem? Please make a backup of your data before testing a development build.

Thanks!

@ThiloteE ThiloteE changed the title v5.2 not responding when Saving library v5.2 High CPU/RAM when saving a large library and field formatters (normalize date) are enabled Jun 16, 2022
@ThiloteE
Copy link
Member

meta-issue: #8906

@ThiloteE
Copy link
Member

To do:

@koppor koppor moved this to High priority in Prioritization Nov 10, 2022
@Siedlerchr Siedlerchr moved this from High priority to Normal priority in Prioritization Mar 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Normal priority
Development

No branches or pull requests

5 participants