Skip to content

Improve volume upload concurrency and progress#1905

Merged
sre-ci-robot merged 1 commit into
masterfrom
feature/volume-sdk-interaction
Jun 24, 2026
Merged

Improve volume upload concurrency and progress#1905
sre-ci-robot merged 1 commit into
masterfrom
feature/volume-sdk-interaction

Conversation

@lentitude2tk

Copy link
Copy Markdown
Collaborator

Summary

  • move Volume upload options to UploadFilesRequest and reduce default file upload concurrency to 5
  • add byte-level upload progress for large files
  • make VolumeFileManager upload state request-local so one client instance can run concurrent uploads safely
  • keep existing public APIs source-compatible

Tests

  • mvn -pl sdk-bulkwriter -DskipTests compile
  • mvn -pl sdk-bulkwriter -Dtest=VolumeManagerTest,ProgressInputStreamTest test

@mergify

mergify Bot commented Jun 23, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

Comment thread sdk-bulkwriter/src/main/java/io/milvus/bulkwriter/VolumeFileManager.java Outdated
Signed-off-by: lentitude2tk <xushuang.hu@zilliz.com>
@lentitude2tk lentitude2tk force-pushed the feature/volume-sdk-interaction branch from 1ac6b88 to 0644b73 Compare June 24, 2026 06:25
@mergify mergify Bot removed the ci-passed label Jun 24, 2026
@yhmo

yhmo commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

/lgtm
/approve

@sre-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lentitude2tk, yhmo

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mergify mergify Bot added the ci-passed label Jun 24, 2026
@sre-ci-robot sre-ci-robot merged commit ee4b440 into master Jun 24, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants