-
Notifications
You must be signed in to change notification settings - Fork 447
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
Improve ScriptHost Function "Swapability" #237
Comments
Is this something which would help mitigate cold start issues? I'm thinking it's only a dev time thing, not necessarily a produciton env thing. Thoughts? |
Yeah, it's mainly a dev time thing. Likely this isn't needed for RC - it's an improvement we can make later if needed. Also, it would be quite a bit of work. |
Cool. Let's punt. |
Closing this. We don't need an open issue to track this. We don't have any plans to do this any time soon, and we can reopen this if/when we decide we need to do this. |
This would require a bunch of work in the core SDK. Currently whenever function metadata changes, it requires us to restart the ScriptHost. This is because in the core SDK, functions are indexed and listeners are started on process startup, and that function set is immutable for the lifetime of the process. That makes sense for classic SDK scenarios.
For Functions however, we'd like more flexibility - we'd like to be able to add/remove functions from the host on the fly, without restarting the host.
The text was updated successfully, but these errors were encountered: