Skip to content

feat(service/obs):add multipart upload function support#2685

Merged
Xuanwo merged 2 commits intoapache:mainfrom
A-Stupid-Sun:feat-obs_write_without_content_length
Jul 21, 2023
Merged

feat(service/obs):add multipart upload function support#2685
Xuanwo merged 2 commits intoapache:mainfrom
A-Stupid-Sun:feat-obs_write_without_content_length

Conversation

@A-Stupid-Sun
Copy link
Copy Markdown
Contributor

@A-Stupid-Sun A-Stupid-Sun commented Jul 21, 2023

Add multipart upload api support for service obs
close issue #2652

@A-Stupid-Sun A-Stupid-Sun requested a review from Xuanwo as a code owner July 21, 2023 09:46
@github-actions github-actions Bot added the releases-note/feat The PR implements a new feature or has a title that begins with "feat" label Jul 21, 2023
@suyanhanx
Copy link
Copy Markdown
Member

suyanhanx commented Jul 21, 2023 •
edited
Loading

Hi, we could add a capability write_without_content_length for this service to enable this test.

https://github.com/apache/incubator-opendal/blob/5edd772da3ea9b0804564a72f177c3c28fc9ff45/core/tests/behavior/write.rs#L1212

@A-Stupid-Sun
Copy link
Copy Markdown
Contributor Author

A-Stupid-Sun commented Jul 21, 2023

Hi, we could add a capability write_without_content_length for this service to enable this test.

https://github.com/apache/incubator-opendal/blob/5edd772da3ea9b0804564a72f177c3c28fc9ff45/core/tests/behavior/write.rs#L1212

done

Xuanwo
Xuanwo approved these changes Jul 21, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thanks a lot!

@Xuanwo Xuanwo merged commit 6c4c5b2 into apache:main Jul 21, 2023
@A-Stupid-Sun A-Stupid-Sun deleted the feat-obs_write_without_content_length branch July 21, 2023 15:50
@suyanhanx suyanhanx mentioned this pull request Jul 23, 2023
Closed
@oowl oowl mentioned this pull request Jul 28, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Xuanwo Xuanwo Xuanwo approved these changes

Assignees

No one assigned

Labels

releases-note/feat The PR implements a new feature or has a title that begins with "feat"

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@A-Stupid-Sun @suyanhanx @Xuanwo