Browse Source

Don't pass empty array to asyncPool when archiving

Simon Detheridge 6 years ago
parent
commit
1575cd41eb
1 changed files with 6 additions and 5 deletions
  1. 6 5
      services/docstore/app/js/DocArchiveManager.js

+ 6 - 5
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) {