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

test: add sync request benchmark #640

Draft
wants to merge 2 commits into
base: tomasarrachea-stress-test
Choose a base branch
from

Conversation

TomasArrachea
Copy link
Collaborator

@TomasArrachea TomasArrachea commented Jan 23, 2025

This PR adds a cli command to execute a sync request and measure the performance. Using the load-state command introduced in 621, the idea is to load the store and then use that dump file to test the sync request under different load conditions.

Also, this PR updates the LoadState command to generate both Public and Private accounts. Additionally it now generates two output files:

  • insertion times measured on loading
  • accounts ids of the public accounts loaded

@TomasArrachea TomasArrachea force-pushed the tomasarrachea-stress-test-sync branch from cf2c27a to 245f159 Compare January 23, 2025 21:41
@bobbinth
Copy link
Contributor

Not a review yet, but I would probably split this up into 2 PRs: one to add more functionality for generating the test data and the other one for benchmarking state sync endpoint performance.

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

Successfully merging this pull request may close these issues.

2 participants