-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Memory problem in Jabref #4323
Comments
The same phenomenon appears in JabRef 5.0-dev--snapshot--2018-09-08 (Debian GNU/Linux). |
Hi, Thanks for your reports, could you please try to tell when exactly the problem occurs? If you want to debug this a bit further, you could try to use the YourKit java Profiler. https://www.yourkit.com/docs/java/help/memory_snapshots.jsp |
Hi, I found an error when opening a new library, which could be the problem here as well. I fixed it as part of another PR. https://builds.jabref.org/storeColumnSortOrder/ |
I'm sorry, man, the https://builds.jabref.org/storeColumnSortOrder/ version doesn't work. I cann't make the JabRef version screenshot( through "Option" > "About") in that the JabRef is freezed. Here's the terminal reads: |
@xiaodongcentury Could you please test again with the latest master version? |
no problem. I'll try it. |
Once i reboot my OS, java -jar xxxx.jar (2018-09-13). Task manager reads: CPU, 100%, |
Hi, I recently had the same problem with it and found that this switch helps: |
OK, Thanks, I'll try your suggestion. |
I have tested the master snapshot of yesterday. (2018-09-13) And I encounter memory problems by a very small excerpt of my database which is included. I suspected an error with unescaped % signs in the abstract, but when I escaped the offending % signs manually using Notepad++, the strange behavior remained. I click on all entry in the file and nothing strange happened. Then I clicked on the entry NS86 I ran the snapshot with the switch proposed, but it does not help. |
after I let run java for a prolonged time I can no longer restart jabref 5dev since an java error occurs:
Please advise what to do. |
Thanks @bernhard-kleine. This narrows down the problem. In fact,
leads to the freeze while
works fine (I just added dots after the first name initials). Moreover, removing the last dot in the working example also results in the freeze. Hence, I guess our integrity checks are the reason for the problem. @bernhard-kleine: system restart? |
These entries were all imported using the PMID Medline search in Jabref. Now that the missing dots are there, I can not open the file which would make Jabref fail immediately. There are some 2660 entries. I cannot solve the problem by manually adding dots. |
I debugged it and traced the problem to impl.org.controlsfx.ImplUtils#injectAsRootPane. For whatever reason, I cannot however step trough this method since the source code shown in the IDE is different from the bytecode. @Siedlerchr can you try to debug it and see where it freezes exactly? |
@tobiasdiez I could trace it down to this line:
which then seems to create the hanging:
|
Fixes #4323 and probably fixes #4294 as well. The reason for the freezes were the pull-request https://bitbucket.org/controlsfx/controlsfx/pull-requests/710. These changes mess up the layout for some reason. The fix is to inject a DecorationPane ourselves directly from the beginning. Although this fix started as a workaround for this bug in controlsfx, I think, we should keep it permanently for performance reasons.
Fixes #4323 and probably fixes #4294 as well. The reason for the freezes were the pull-request https://bitbucket.org/controlsfx/controlsfx/pull-requests/710. These changes mess up the layout for some reason. The fix is to inject a DecorationPane ourselves directly from the beginning. Although this fix started as a workaround for this bug in controlsfx, I think, we should keep it permanently for performance reasons.
I can confirm that the latest development version with the fix from @tobiasdiez now works fine! the memory footprint is much lower now and no more freezes in entry editor. |
I confirm that the import reported in #4345 is now flawlessly. |
JabRef 5.0-dev--snapshot--2018-09-06--docbookfive--ee248ae75
Windows 7 6.1 amd64
Java 1.8.0_181
Steps to reproduce:
Jabref chokes and the program can only be shut down from the task manager. It only increases available RAM up to 4.4 Gb (and even further, but I did not wait for it) without doing anything. A log file or error report is not possible.
The text was updated successfully, but these errors were encountered: