nextcloud-libraries / nextcloud-libraries/nextcloud-upload

MOVE should be retryable

Open
#1,697 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

0. Needs triage technical debt
Dominant language
TypeScript
Stars
10
Forks
5
Avg merge
2d 10h
Merged PRs (30d)
14

Description

When uploading a large file (~4GB), the upload is splitted to chunks. Each chunk (generally ~10MB) are small enough so that if there is a network error, only the chunk in error are reuploaded.

At the end of upload, a MOVE operation is done to reassemble the chunks. This operation can take a long time (~3min) and can fail . Shouldn't we make the MOVE retryable (3 time)?

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the upload entry point that performs the final MOVE operation and compare its error handling with the existing chunk-upload retries. Confirm the behavior with a failed or delayed MOVE, and consider the work done when the operation retries up to three times before reporting failure.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
api
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.