mirror of
https://github.com/overleaf/overleaf.git
synced 2024-11-21 20:47:08 -05:00
Merge pull request #4381 from overleaf/tm-history-upgrade-batched-update-hints
Extend batchedUpdate to accept find() options, and use a hint in upgrade script query to suggest mongo uses the _id index GitOrigin-RevId: 7115f84d8be0c78ccb443051e911c931bf4aa0de
This commit is contained in:
parent
99fe2eca51
commit
e25d8b1440
2 changed files with 24 additions and 6 deletions
|
@ -7,13 +7,13 @@ if (process.env.BATCH_LAST_ID) {
|
|||
BATCH_LAST_ID = ObjectId(process.env.BATCH_LAST_ID)
|
||||
}
|
||||
|
||||
async function getNextBatch(collection, query, maxId, projection) {
|
||||
async function getNextBatch(collection, query, maxId, projection, options) {
|
||||
maxId = maxId || BATCH_LAST_ID
|
||||
if (maxId) {
|
||||
query._id = { $gt: maxId }
|
||||
}
|
||||
const entries = await collection
|
||||
.find(query, { readPreference: ReadPreference.SECONDARY })
|
||||
.find(query, options)
|
||||
.project(projection)
|
||||
.sort({ _id: 1 })
|
||||
.limit(BATCH_SIZE)
|
||||
|
@ -28,17 +28,31 @@ async function performUpdate(collection, nextBatch, update) {
|
|||
)
|
||||
}
|
||||
|
||||
async function batchedUpdate(collectionName, query, update, projection) {
|
||||
async function batchedUpdate(
|
||||
collectionName,
|
||||
query,
|
||||
update,
|
||||
projection,
|
||||
options
|
||||
) {
|
||||
await waitForDb()
|
||||
const collection = db[collectionName]
|
||||
|
||||
options = options || {}
|
||||
options.readPreference = ReadPreference.SECONDARY
|
||||
|
||||
projection = projection || { _id: 1 }
|
||||
let nextBatch
|
||||
let updated = 0
|
||||
let maxId
|
||||
while (
|
||||
(nextBatch = await getNextBatch(collection, query, maxId, projection))
|
||||
.length
|
||||
(nextBatch = await getNextBatch(
|
||||
collection,
|
||||
query,
|
||||
maxId,
|
||||
projection,
|
||||
options
|
||||
)).length
|
||||
) {
|
||||
maxId = nextBatch[nextBatch.length - 1]._id
|
||||
updated += nextBatch.length
|
||||
|
|
|
@ -79,6 +79,9 @@ async function main() {
|
|||
_id: 1,
|
||||
overleaf: 1,
|
||||
}
|
||||
const options = {
|
||||
hint: { _id: 1 },
|
||||
}
|
||||
await batchedUpdate(
|
||||
'projects',
|
||||
{
|
||||
|
@ -88,7 +91,8 @@ async function main() {
|
|||
],
|
||||
},
|
||||
processBatch,
|
||||
projection
|
||||
projection,
|
||||
options
|
||||
)
|
||||
console.log('Final')
|
||||
console.log(RESULT)
|
||||
|
|
Loading…
Reference in a new issue