Skip to content

Comments

RFC-5479: Context#5479

Merged
Xuanwo merged 2 commits intomainfrom
rfc-context
Dec 31, 2024
Merged

RFC-5479: Context#5479
Xuanwo merged 2 commits intomainfrom
rfc-context

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 PsiACE and meteorgan 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