Skip to content

[IN-173]: feat/remove-folder-upload-wait - #2205

Merged
sg-gs merged 1 commit into
masterfrom
feat/IN-173-remove-folder-upload-wait
Oct 8, 2026
Merged

sg-gs merged 1 commit into
masterfrom
feat/IN-173-remove-folder-upload-wait

Conversation

@sg-gs

@sg-gs sg-gs commented Oct 7, 2026

Copy link
Copy Markdown
Member

What

Folder uploads no longer wait a fixed time after creating each folder (600 ms in UploadFolderManager, 500 ms in uploadFolderThunk). The wait is kept only when a workspace is selected.

Why

The wait only existed so the backend would see the new folder. internxt/drive-server-wip#1203 now validates the parent folder against the primary database, so it is no longer needed. According to the upload benchmark (internxt/infra#34), it cost ~25% of the time on uploads with many small folders. Workspace endpoints still validate against the read replica, so workspace uploads keep the wait.

@sg-gs sg-gs added the enhancement New feature or request label Oct 7, 2026
@sg-gs sg-gs self-assigned this Oct 7, 2026
@sg-gs
sg-gs requested a review from CandelR October 7, 2026 15:33
@sg-gs
sg-gs marked this pull request as ready for review October 7, 2026 15:36
@sg-gs
sg-gs requested review from a team and larryrider as code owners October 7, 2026 15:36
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: b7b2901
Status: ✅  Deploy successful!
Preview URL: https://3a8ecae8.drive-web.pages.dev
Branch Preview URL: https://feat-in-173-remove-folder-up.drive-web.pages.dev

View logs

@sonarqubecloud

sonarqubecloud Bot commented Oct 7, 2026

Copy link
Copy Markdown

@sg-gs
sg-gs merged commit 0d79cce into master Oct 8, 2026
11 checks passed
@sg-gs
sg-gs deleted the feat/IN-173-remove-folder-upload-wait branch October 8, 2026 05:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants