github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2024-11-07 20:31:06 -05:00
Code Issues Projects Releases Packages Wiki Activity

log for each get/try/release lock

Browse source
This commit is contained in:
Brian Gough 2017-05-17 16:11:48 +01:00
parent 4622362e42
commit 387ecac6cb
1 changed files with 9 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
services/document-updater/app/coffee/LockManager.coffee
View file

@ -30,14 +30,16 @@ module.exports = LockManager =
tryLock : (doc_id, callback = (err, isFree)->)->
lockValue = LockManager.randomLock()
key = keys.blockingKey(doc_id:doc_id)
logger.log {doc_id: doc_id, key: key}, "tryLock"
rclient.set key, lockValue, "EX", @LOCK_TTL, "NX", (err, gotLock)->
return callback(err) if err?
if gotLock == "OK"
metrics.inc "doc-not-blocking"
logger.log {doc_id: doc_id, key: key, lockValue: lockValue}, "got lock"
callback err, true, lockValue
else
metrics.inc "doc-blocking"
logger.log {doc_id}, "doc is locked"
logger.log {doc_id: doc_id, key: key}, "doc is locked"
callback err, false
getLock: (doc_id, callback = (error, lockValue) ->) ->
@ -54,6 +56,7 @@ module.exports = LockManager =
if gotLock
callback(null, lockValue)
else
logger.log {doc_id: doc_id, delay: testInterval}, "will retry lock"
setTimeout attempt, testInterval
# back off when the lock is taken to avoid overloading
testInterval = Math.min(testInterval * 2, LockManager.MAX_TEST_INTERVAL)
@ -75,7 +78,9 @@ module.exports = LockManager =
rclient.eval LockManager.unlockScript, 1, key, lockValue, (err, result) ->
if err?
return callback(err)
if result? and result isnt 1 # successful unlock should release exactly one key
logger.error {doc_id:doc_id, lockValue:lockValue, redis_err:err, redis_result:result}, "unlocking error"
else if result? and result isnt 1 # successful unlock should release exactly one key
logger.error {doc_id:doc_id, key:key, lockValue:lockValue, redis_err:err, redis_result:result}, "unlocking error"
return callback(new Error("tried to release timed out lock"))
callback(err,result)
else
logger.log {doc_id:doc_id, key:key, lockValue:lockValue}, "released lock"
callback(err,result)