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

Issue #1076: load commit cache in a background thread #1140

Merged
merged 1 commit into from Oct 27, 2016

Conversation

tomaswolf
Copy link
Contributor

@tomaswolf tomaswolf commented Oct 27, 2016

  • Make the CommitCache fully thread-safe. It was using a
    ConcurrentHashMap containing lists, but then handed out these lists.
    It also did multiple operations on that map that as a whole should
    be atomic.
  • Use isEmpty() instead of size() == 0.
  • Run the loading of the commit cache in a background daemon thread

* Make the CommitCache fully thread-safe. It was using a
  ConcurrentHashMap containing lists, but then handed out these lists.
  It also did multiple operations on that map that as a whole should
  be atomic.
  
* Use isEmpty() instead of size() == 0.

* Run the loading of the commit cache in a background daemon thread
@gitblit gitblit merged commit bac1241 into gitblit:master Oct 27, 2016
@gitblit
Copy link
Owner

@gitblit gitblit commented Oct 27, 2016

👍 Looks great!

@tomaswolf tomaswolf deleted the issue_1076 branch Oct 31, 2016
@flaix flaix added this to the 1.9.0 milestone Mar 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants