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.

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 ZkTracer initialization time #11

Merged
merged 4 commits into from
May 16, 2024
Merged

Conversation

ahamlat
Copy link
Contributor

@ahamlat ahamlat commented May 16, 2024

When doing load testing on eth_sendRawTransaction with transaction simulation enabled, we noticed that ZkTracer takes more than 20% of the time, on loading the opcode and the spillings.

image

This RP Improves ZkTracer initialization time by doing only once Opcodes and spillings loading from disk resources. This is done when the class is loaded, opcodes and spillings are initialized is a static way.

…pilling loading from disk resources

Signed-off-by: Ameziane H <[email protected]>
@ahamlat ahamlat requested a review from fab-10 May 16, 2024 09:08
Signed-off-by: Ameziane H <[email protected]>
ahamlat added 2 commits May 16, 2024 15:38
Signed-off-by: Ameziane H <[email protected]>
@ahamlat ahamlat merged commit 764914e into main May 16, 2024
7 checks passed
@fab-10 fab-10 deleted the improve-Zktracer-init-time branch June 26, 2024 14:16
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.

3 participants