Skip to content

Conversation

@jonathanc-n
Copy link
Contributor

Which issue does this PR close?

Part of #5693

Rationale for this change

Return metadata for write.

What changes are included in this PR?

I had copied what @meteorgan did with the azblob change (thanks! 😄 ). Just trying to learn some of the internals for services in Opendal

Are there any user-facing changes?

@jonathanc-n jonathanc-n requested a review from Xuanwo as a code owner July 6, 2025 01:25
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. releases-note/feat The PR implements a new feature or has a title that begins with "feat" labels Jul 6, 2025
@jonathanc-n
Copy link
Contributor Author

@Xuanwo How come it automatically request a review?

@Xuanwo
Copy link
Member

Xuanwo commented Jul 6, 2025

@Xuanwo How come it automatically request a review?

I'm the code owner for core 😆

Copy link
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.

Thank you @jonathanc-n for working on this!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 6, 2025
@Xuanwo Xuanwo merged commit 27d5677 into apache:main Jul 6, 2025
87 checks passed
@Xuanwo Xuanwo mentioned this pull request Jul 11, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/feat The PR implements a new feature or has a title that begins with "feat" size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants