From 1575cd41ebf8ac5deccf199ec8121e6020f23011 Mon Sep 17 00:00:00 2001 From: Simon Detheridge Date: Mon, 10 Aug 2020 15:44:41 +0100 Subject: [PATCH] Don't pass empty array to asyncPool when archiving --- services/docstore/app/js/DocArchiveManager.js | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/services/docstore/app/js/DocArchiveManager.js b/services/docstore/app/js/DocArchiveManager.js index e7fc4ee848..d564113c9b 100644 --- a/services/docstore/app/js/DocArchiveManager.js +++ b/services/docstore/app/js/DocArchiveManager.js @@ -39,11 +39,12 @@ async function archiveAllDocs(projectId) { throw new Errors.NotFoundError(`No docs for project ${projectId}`) } - await asyncPool( - PARALLEL_JOBS, - docs.filter((doc) => !doc.inS3), - (doc) => archiveDoc(projectId, doc) - ) + const docsToArchive = docs.filter((doc) => !doc.inS3) + if (docsToArchive.length) { + await asyncPool(PARALLEL_JOBS, docsToArchive, (doc) => + archiveDoc(projectId, doc) + ) + } } async function archiveDoc(projectId, doc) {