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.

Sign up for GitHub

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

Jump to bottom

[Schemas] Add caching to the schema.Get() function #158

Closed
hariso opened this issue Jul 30, 2024 · 0 comments · Fixed by #162
Closed

[Schemas] Add caching to the schema.Get() function #158

hariso opened this issue Jul 30, 2024 · 0 comments · Fixed by #162
Labels
feature New feature or request triage Needs to be triaged

Comments

@hariso
Copy link
Contributor

hariso commented Jul 30, 2024

Feature description

In standalone plugins, schema.Get() uses a gRPC call to get a schema. To avoid the cost of repeatedly fetching or creating the same schema many times (especially over gRPC), schemas should be cached by the SDK.

Reference: https://github.com/ConduitIO/conduit/blob/main/docs/design-documents/20240430-schema-support.md#requirements
@hariso hariso added feature New feature or request triage Needs to be triaged labels Jul 30, 2024
@hariso hariso mentioned this issue Jul 30, 2024
Closed
17 tasks
@lovromazgon lovromazgon mentioned this issue Aug 1, 2024
Merged
4 tasks
@github-project-automation github-project-automation bot moved this from Triage to Done in Conduit Main Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature New feature or request triage Needs to be triaged
Projects
Conduit Main
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow provided schema, schema cache ConduitIO/conduit-connector-sdk
1 participant
@hariso