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

RFC-5479: Context #5479

Merged
merged 2 commits into from
Dec 31, 2024
Merged

RFC-5479: Context #5479

merged 2 commits into from
Dec 31, 2024

Conversation

Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Dec 30, 2024

Which issue does this PR close?

Part of #5325

Rationale for this change

This RFC will add Context in opendal to distribute global resources like http client, runtime, etc.

What changes are included in this PR?

A new RFC.

Are there any user-facing changes?

New API.

Signed-off-by: Xuanwo <[email protected]>
@Xuanwo Xuanwo changed the title rfc: Context RFC-5479: Context Dec 30, 2024
Signed-off-by: Xuanwo <[email protected]>
@Xuanwo Xuanwo requested review from meteorgan and PsiACE December 30, 2024 14:20
@meteorgan
Copy link
Contributor

LGTM

@Xuanwo
Copy link
Member Author

Xuanwo commented Dec 31, 2024

LGTM

Hi, please approve this PR if it looks good to you.

@Xuanwo Xuanwo merged commit b6fdd28 into main Dec 31, 2024
262 of 263 checks passed
@Xuanwo Xuanwo deleted the rfc-context branch December 31, 2024 15:33
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.

2 participants