Resolve "`sync` uses too much RAM, even without GVA enabled"
Closes #1430 (closed)
Edited by Cédric Moreau
Merge request reports
Activity
added 1 commit
- 88affa10 - fix: #1430 (closed) sync is keeping chunks in memory
added 1 commit
- 511d2047 - fix: #1430 (closed) sync is keeping chunks in memory
requested review from @librelois
Pour commenter un peu : en fait, l'instruction
delete this.chunks[this.currentChunkNumber];
retirait bien chaque chunk de la mémoire au fur et à mesure de l'avancée de la synchronisation, sauf que l'instructionconst previous = await this.downloadChunk(i - 1);
utilisée pour vérifier le chaînage ... re-téléchargeait la chunk, et la remettait en mémoire.Du coup, là, j'ai bien vérifié, les chunks sont bien consommées au fur et à mesure et ne sont pas re-téléchargées.
On gagne aussi en téléchargement, car on ne re-télécharge pas deux fois la même chunk.
Je te laisse merger @librelois
mentioned in commit 66e5fc2e
Please register or sign in to reply