Skip to content

Improve perfomance of opening a database that still used the old groups format#2636

Merged
tobiasdiez merged 2 commits into
masterfrom
fixSave
Mar 13, 2017
Merged

Improve perfomance of opening a database that still used the old groups format#2636
tobiasdiez merged 2 commits into
masterfrom
fixSave

Conversation

@tobiasdiez

@tobiasdiez tobiasdiez commented Mar 12, 2017

Copy link
Copy Markdown
Member

Previously, the conversion of the old groups format to the new one was performed after the database was completely loaded and displayed. Thus after every change during the conversion the groups panel and main table were redrawn, resulting in performance problems (in particular with larger db's).
Now the conversion is done before the db is displayed (and thus nobody yet listens to the changes).

Fixes: http://discourse.jabref.org/t/v3-8-2-x64-windows-problem-saving-large-bib-libraries/456

  • Change in CHANGELOG.md described
  • Tests created for changes
  • Screenshots added (for bigger UI changes)
  • Manually tested changed features in running JabRef
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)
  • If you changed the localization: Did you run gradle localizationUpdate?

@tobiasdiez tobiasdiez added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Mar 12, 2017
@tobiasdiez tobiasdiez merged commit 55f6a39 into master Mar 13, 2017
@tobiasdiez tobiasdiez deleted the fixSave branch March 13, 2017 11:34
Siedlerchr added a commit that referenced this pull request Mar 14, 2017
* upstream/master:
  Donate to JabRef!
  Issuestats.com is down
  Add link to logo creator
  Improve perfomance of opening a database that still used the old groups format (#2636)
  Set POSIX filepermissions to 644 (#2637)
  Fix #2629: Removing entry from group updates view properly
  Remove empty tooltip in group panel
  Refactored around the PDF Annotation Importer (#2624)
  Fix last import issue
  Fix imports and some reformatting
  Fix wrong import order
  Add missing svg icon
  RIP old icon
  Add new logo files (#2164)

# Conflicts:
#	src/main/java/org/jabref/gui/groups/GroupSelector.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants