Skip to content

refactor(services/oss): Refactor oss_put_object signatures by using OpWrite#3080

Merged
Xuanwo merged 2 commits intoapache:mainfrom
sysu-yunz:dev/refactor-op-in-oss
Sep 15, 2023
Merged

refactor(services/oss): Refactor oss_put_object signatures by using OpWrite#3080
Xuanwo merged 2 commits intoapache:mainfrom
sysu-yunz:dev/refactor-op-in-oss

Conversation

@sysu-yunz
Copy link
Copy Markdown
Contributor

@sysu-yunz sysu-yunz commented Sep 15, 2023

#3064

  • Refactor oss_put_object oss_put_object_request function in services/oss core.rs
  • Usage of oss_put_object_request in writer.rs and backend.rs also changed

@sysu-yunz sysu-yunz requested a review from Xuanwo as a code owner September 15, 2023 02:43
@github-actions github-actions Bot added the releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" label Sep 15, 2023
@sysu-yunz sysu-yunz changed the title refactor(services/oss): Refactor oss_put_object signatures by using OpWrite Refactor(services/oss): Refactor oss_put_object signatures by using OpWrite Sep 15, 2023
@Xuanwo Xuanwo changed the title Refactor(services/oss): Refactor oss_put_object signatures by using OpWrite refactor(services/oss): Refactor oss_put_object signatures by using OpWrite Sep 15, 2023
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 d797f05 into apache:main Sep 15, 2023
@sysu-yunz sysu-yunz deleted the dev/refactor-op-in-oss branch September 15, 2023 03:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants