Drop lockedCandidatesForBatch and mark chunks as 'busy' when prepar…

…ing (#3342)
transloadit · Dec 7, 2021 · a6270bf · a6270bf
1 parent 10140cb
commit a6270bf
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/packages/@uppy/aws-s3-multipart/src/MultipartUploader.js b/packages/@uppy/aws-s3-multipart/src/MultipartUploader.js
@@ -58,7 +58,6 @@ class MultipartUploader {
     this.partsInProgress = 0
     this.chunks = null
     this.chunkState = null
-    this.lockedCandidatesForBatch = []
 
     this.#initChunks()
 
@@ -181,8 +180,6 @@ class MultipartUploader {
 
     const candidates = []
     for (let i = 0; i < this.chunkState.length; i++) {
-      // eslint-disable-next-line no-continue
-      if (this.lockedCandidatesForBatch.includes(i)) continue
       const state = this.chunkState[i]
       // eslint-disable-next-line no-continue
       if (state.done || state.busy) continue
@@ -242,7 +239,9 @@ class MultipartUploader {
   }
 
   async #prepareUploadParts (candidates) {
-    this.lockedCandidatesForBatch.push(...candidates)
+    candidates.forEach((i) => {
+      this.chunkState[i].busy = true
+    })
 
     const result = await this.#retryable({
       attempt: () => this.options.prepareUploadParts({