Page MenuHomePhabricator

Use file.allocate to upload large files from `arc diff`
ClosedPublic

Authored by epriestley on May 26 2015, 3:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Feb 12, 9:21 PM
Unknown Object (File)
Sun, Feb 9, 7:06 PM
Unknown Object (File)
Feb 8 2025, 4:53 AM
Unknown Object (File)
Feb 8 2025, 4:53 AM
Unknown Object (File)
Feb 8 2025, 4:53 AM
Unknown Object (File)
Feb 8 2025, 4:53 AM
Unknown Object (File)
Jan 25 2025, 9:15 PM
Unknown Object (File)
Jan 24 2025, 8:17 AM
Subscribers
Tokens
"Mountain of Wealth" token, awarded by btrahan.

Details

Summary

Fixes T8259. Depends on D13016. Use modern logic to support large file uploads.

Test Plan
  • Diffed with some images, saw them show up in the diff.
  • Diffed with a 12MB binary, saw it upload in chunks.

Diff Detail

Repository
rARC Arcanist
Branch
upload2
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 6285
Build 6307: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Use file.allocate to upload large files from `arc diff`.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan awarded a token.
btrahan edited edge metadata.
This revision is now accepted and ready to land.May 27 2015, 5:05 PM
This revision was automatically updated to reflect the committed changes.