Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: support resumable upload #50

Merged
merged 5 commits into from
Jun 28, 2023
Merged

feat: support resumable upload #50

merged 5 commits into from
Jun 28, 2023

Conversation

jingjunLi
Copy link
Contributor

@jingjunLi jingjunLi commented Jun 27, 2023

Description

1.Support resumableUpload flag in put command
2. update dependency

Rationale

tell us why we need these changes...

Example

NA

Changes

Notable changes:

  • add each change in a bullet point here
  • ...

@jingjunLi jingjunLi changed the base branch from master to develop June 27, 2023 11:19
cmd/utils.go Outdated

// resumable download & upload
partSizeFlag = "partSize"
disableResumableFlag = "disable-resumable"
Copy link
Collaborator

@flywukong flywukong Jun 27, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggest to: change to MultipartFlag = "multipartUpload"

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can change to support by default after the API tested stable

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

default multipartUpload = false , not use resumble API

Copy link
Contributor Author

@jingjunLi jingjunLi Jun 28, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Fixed disableResumableFlag = "disable-resumable" to resumableUploadFlag = "resumableUpload"
  2. By default, the resumable upload feature is disabled, indicated by the value false.

@flywukong flywukong merged commit cfc5931 into develop Jun 28, 2023
@jingjunLi jingjunLi changed the title feat: resumable upload & upload feat: support resumable upload Jun 28, 2023
@flywukong flywukong deleted the feat-resumable-upload branch October 13, 2023 02:22
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.

3 participants