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

Don't store the OpenAPI contract anymore, rather just cache it #2337

Merged

slinkydeveloper merged 1 commit into restatedev:main from slinkydeveloper:lazy-openapi

Nov 22, 2024

+22 −21

Contributor

slinkydeveloper commented Nov 21, 2024

For now we put this cache inside the ServiceSchemas data structure, but in future we should move it somewhere else (perhaps in Schemas)


          Don't store the OpenAPI contract anymore, rather just cache it

f12dd38

slinkydeveloper requested a review from tillrohrmann

November 21, 2024 15:29

tillrohrmann approved these changes

View reviewed changes

Contributor

tillrohrmann left a comment

LGTM. +1 for merging.

slinkydeveloper merged commit f846f1c into restatedev:main

10 checks passed

slinkydeveloper deleted the lazy-openapi branch

November 22, 2024 11:36

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet