Skip to content

Comments

feat: Add HttpClientLayer to replace Operator::update_http_client()#6290

Merged
Xuanwo merged 5 commits intomainfrom
add-http-client-layer
Jun 13, 2025
Merged

feat: Add HttpClientLayer to replace Operator::update_http_client()#6290
Xuanwo merged 5 commits intomainfrom
add-http-client-layer

Conversation

@Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Jun 13, 2025

Which issue does this PR close?

None

Rationale for this change

Make OpenDAL's API more consistent.

What changes are included in this PR?

Add HttpClientLayer to replace Operator::update_http_client()

Are there any user-facing changes?

Deprecate existing APIs.

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jun 13, 2025
@dosubot dosubot bot added the releases-note/feat The PR implements a new feature or has a title that begins with "feat" label Jun 13, 2025
Copy link
Contributor

@yihong0618 yihong0618 left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Xuanwo <[email protected]>
@Xuanwo Xuanwo merged commit 2c0aed8 into main Jun 13, 2025
359 checks passed
@Xuanwo Xuanwo deleted the add-http-client-layer branch June 13, 2025 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/feat The PR implements a new feature or has a title that begins with "feat" size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants