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.

Sign up for GitHub

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

Jump to bottom

orm: Improves locking behavior #67

Merged
merged 2 commits into from
Sep 17, 2015
Merged

orm: Improves locking behavior #67

merged 2 commits into from
Sep 17, 2015

Conversation

Koed00
Copy link
Owner

@Koed00 Koed00 commented Sep 17, 2015

  • Improves the locking behavior of the orm broker
  • Adds lock counter to the monitor for the orm broker

@Koed00
Changed locking mechanism
1d12107 
* all new tasks start as timeouts. This makes the queries a lot simpler.

* In BULK mode, the bulk update for the lock was causing actual database locks.  The lock updates are now iterated.
@Koed00
Adds lock count to monitor for orm broker
a6ad47b
@codecov-io
Copy link

Current coverage is 95.20%

Merging #67 into master will increase coverage by +0.15% as of dc2e7a7

@@            master    #67   diff @@
=====================================
  Files           36     36       
  Stmts         2002   2022    +20
  Branches         0      0       
  Methods          0      0       
=====================================
+ Hit           1903   1925    +22
  Partial          0      0       
+ Missed          99     97     -2

Review entire Coverage Diff as of dc2e7a7

Powered by Codecov. Updated on successful CI builds.

Koed00 added a commit that referenced this pull request Sep 17, 2015
@Koed00
Merge pull request #67 from Koed00/dev
0d335d2 
orm: Improves locking behavior
@Koed00 Koed00 merged commit 0d335d2 into master Sep 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Koed00 @codecov-io