Skip to content

Comments

RFC-5444: Operator From Uri#5444

Merged
Xuanwo merged 6 commits intomainfrom
operator-from-uri
Dec 27, 2024
Merged

RFC-5444: Operator From Uri#5444
Xuanwo merged 6 commits intomainfrom
operator-from-uri

Conversation

@Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Dec 23, 2024

Which issue does this PR close?

Closes #5407

Rationale for this change

This RFC proposes adding URI-based configuration support to OpenDAL, allowing users to create operators directly from URIs. The proposal introduces a new from_uri API in both the Operator and Configurator traits, along with an OperatorRegistry to manage operator factories.

What changes are included in this PR?

New RFC

Are there any user-facing changes?

Signed-off-by: Xuanwo <[email protected]>
@Xuanwo Xuanwo changed the title rfc: Operator From Uri RFC-5444: Operator From Uri Dec 23, 2024
Signed-off-by: Xuanwo <[email protected]>
Signed-off-by: Xuanwo <[email protected]>
Signed-off-by: Xuanwo <[email protected]>
Signed-off-by: Xuanwo <[email protected]>
Copy link
Member

@Zheaoli Zheaoli left a comment

Choose a reason for hiding this comment

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

LGTM

@Xuanwo
Copy link
Member Author

Xuanwo commented Dec 27, 2024

Thank you all for the review, let's move!

@Xuanwo Xuanwo merged commit 9dbef16 into main Dec 27, 2024
261 checks passed
@Xuanwo Xuanwo deleted the operator-from-uri branch December 27, 2024 02:49
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.

Support creating operator from URL

5 participants