github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2024-12-24 05:11:49 +00:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #247 from overleaf/jpa-track-db-times

Browse source 
[misc] track delays of using sqlite in a few places
This commit is contained in:
Jakob Ackermann 2021-06-15 10:11:46 +02:00 committed by GitHub
commit 8caa19afed
2 changed files with 27 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
services/clsi/app/js/ProjectPersistenceManager.js
View file

@ -12,6 +12,7 @@
* Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
*/
let ProjectPersistenceManager
const Metrics = require('./Metrics')
const UrlCache = require('./UrlCache')
const CompileManager = require('./CompileManager')
const db = require('./db')
@ -64,6 +65,7 @@ module.exports = ProjectPersistenceManager = {
if (callback == null) {
callback = function (error) {}
}
const timer = new Metrics.Timer('db-bump-last-accessed')
const job = (cb) =>
db.Project.findOrCreate({ where: { project_id } })
.spread((project, created) =>
@ -73,7 +75,10 @@ module.exports = ProjectPersistenceManager = {
.error(cb)
)
.error(cb)
return dbQueue.queue.push(job, callback)
dbQueue.queue.push(job, (error) => {
timer.done()
callback(error)
})
},
clearExpiredProjects(callback) {

25
services/clsi/app/js/UrlCache.js
View file

@ -21,6 +21,7 @@ const crypto = require('crypto')
const fs = require('fs')
const logger = require('logger-sharelatex')
const async = require('async')
const Metrics = require('./Metrics')
module.exports = UrlCache = {
downloadUrlToFile(project_id, url, destPath, lastModified, callback) {
@ -206,17 +207,22 @@ module.exports = UrlCache = {
if (callback == null) {
callback = function (error, urlDetails) {}
}
const timer = new Metrics.Timer('db-find-url-details')
const job = (cb) =>
db.UrlCache.findOne({ where: { url, project_id } })
.then((urlDetails) => cb(null, urlDetails))
.error(cb)
return dbQueue.queue.push(job, callback)
dbQueue.queue.push(job, (error, urlDetails) => {
timer.done()
callback(error, urlDetails)
})
},
_updateOrCreateUrlDetails(project_id, url, lastModified, callback) {
if (callback == null) {
callback = function (error) {}
}
const timer = new Metrics.Timer('db-update-or-create-url-details')
const job = (cb) =>
db.UrlCache.findOrCreate({ where: { url, project_id } })
.spread((urlDetails, created) =>
@ -226,24 +232,32 @@ module.exports = UrlCache = {
.error(cb)
)
.error(cb)
return dbQueue.queue.push(job, callback)
dbQueue.queue.push(job, (error) => {
timer.done()
callback(error)
})
},
_clearUrlDetails(project_id, url, callback) {
if (callback == null) {
callback = function (error) {}
}
const timer = new Metrics.Timer('db-clear-url-details')
const job = (cb) =>
db.UrlCache.destroy({ where: { url, project_id } })
.then(() => cb(null))
.error(cb)
return dbQueue.queue.push(job, callback)
dbQueue.queue.push(job, (error) => {
timer.done()
callback(error)
})
},
_findAllUrlsInProject(project_id, callback) {
if (callback == null) {
callback = function (error, urls) {}
}
const timer = new Metrics.Timer('db-find-urls-in-project')
const job = (cb) =>
db.UrlCache.findAll({ where: { project_id } })
.then((urlEntries) =>
@ -253,6 +267,9 @@ module.exports = UrlCache = {
)
)
.error(cb)
return dbQueue.queue.push(job, callback)
dbQueue.queue.push(job, (err, urls) => {
timer.done()
callback(err, urls)
})
}
}