-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Snapshots: move some of the bootstrap options from runtime to snapshot time #21133
Comments
littledivy
added a commit
that referenced
this issue
Nov 13, 2023
Move most runtime options to be lazily loaded. Constant options will be covered in a different PR. Towards #21133
littledivy
added a commit
that referenced
this issue
Nov 15, 2023
kt3k
pushed a commit
that referenced
this issue
Nov 17, 2023
Move most runtime options to be lazily loaded. Constant options will be covered in a different PR. Towards #21133
kt3k
pushed a commit
that referenced
this issue
Nov 17, 2023
zifeo
pushed a commit
to metatypedev/deno
that referenced
this issue
Nov 22, 2023
Move most runtime options to be lazily loaded. Constant options will be covered in a different PR. Towards denoland#21133
zifeo
pushed a commit
to metatypedev/deno
that referenced
this issue
Nov 22, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We have following data that needs to be sent over to the runtime during bootstrapping:
deno/runtime/worker_bootstrap.rs
Lines 109 to 148 in 9010b8d
Some of this data is fixed for a particular build of Deno, namely:
runtime_version
ts_version
env!("TARGET")
v8_version
user_agent
All of these are strings which are the most expensive stuff to serialize on bootstrap. Moving them to be defined only once during snapshot should help us (although minimally) to improve the startup time.
The text was updated successfully, but these errors were encountered: