Skip to content

feat!: Remove multipart uploads - #701

Closed
lcian wants to merge 1 commit into
mainfrom
chore/remove-multipart-uploads
Closed

lcian wants to merge 1 commit into
mainfrom
chore/remove-multipart-uploads

Conversation

@lcian

@lcian lcian commented Oct 21, 2026

Copy link
Copy Markdown
Member

Remove the multipart upload APIs from the Rust and Python clients together with the server routes, shared protocol types, and service/backend interfaces. Delete their tests, documentation, stress-test support, and tiered completion cleanup state. The long-term tier now uses Backend directly and LongTermStorageConfig keeps the existing configuration format. Batch requests and ordinary GCS writes retain their HTTP multipart encoding.

This breaks the multipart client API and removes the objects:multipart* routes. Consumers must migrate to resumable uploads, and deployed clients must be updated to 0.3.0 before deploying the server removal, as required by FS-475.

Local Rust and Python checks pass, including the remaining client/server tests. The full workspace test run is limited by 67 emulator-dependent service failures: Docker reports storage I/O errors and cannot start the backend emulators.

Closes FS-475

Remove the upload API, service and backend interfaces, protocol types, tests, documentation, and stress-test support.

BREAKING CHANGE: Multipart upload client APIs and server routes are removed.

Refs FS-475
@linear-code

linear-code Bot commented Oct 21, 2026

Copy link
Copy Markdown

FS-475

@codecov

codecov Bot commented Oct 21, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 54.54545% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.27%. Comparing base (9c859ab) to head (c13bd66).

Files with missing lines Patch % Lines
objectstore-service/src/backend/mod.rs 0.00% 4 Missing ⚠️
objectstore-test/src/server.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #701      +/-   ##
==========================================
- Coverage   91.40%   91.27%   -0.14%     
==========================================
  Files         118      114       -4     
  Lines       24551    21785    -2766     
==========================================
- Hits        22441    19884    -2557     
+ Misses       2110     1901     -209     
Components Coverage Δ
Rust Backend 94.59% <55.55%> (-0.28%) ⬇️
Rust Client 82.38% <ø> (+0.55%) ⬆️
Python Client 93.51% <ø> (+0.23%) ⬆️

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lcian lcian closed this Oct 21, 2026
@lcian lcian reopened this Oct 21, 2026
@lcian lcian changed the title feat!: Remove multipart uploads from clients and server feat!: Remove multipart uploads Oct 21, 2026
@lcian lcian closed this Oct 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant