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

Simplify scalability chart #2134

Merged
merged 2 commits into from
Dec 9, 2024
Merged

Conversation

KyleAMathews
Copy link
Contributor

No description provided.

Copy link
Contributor

@thruflo thruflo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great 👍

One thing is that it would be nice to still display what the write workload is. Even if we pick one, it’s still useful to know that this is with a realistic write workload going on.

The other thing that we lose is the CPU. Perhaps we can just plot that on a separate graph on the actual benchmarks page. But for a summary for both blog post and benchmarks page to start with this is great to go ahead with.

@KyleAMathews
Copy link
Contributor Author

oh right yeah — the 960 writes / minute is important context — I'll add that back in. It'd be interesting too to list how many bytes / second that is at the 1 million client mark e.g. if the transactions are 5kb (fairly small updates) that's 80 GB / second we'd be pushing out.

@KyleAMathews
Copy link
Contributor Author

@magnetised do you know how big the transactions or shape updates are more-or-less?

@KyleAMathews KyleAMathews merged commit 668242e into thruflo/beta-release-post Dec 9, 2024
2 checks passed
@KyleAMathews KyleAMathews deleted the chart-revision branch December 9, 2024 22:14
thruflo pushed a commit that referenced this pull request Dec 10, 2024
thruflo pushed a commit that referenced this pull request Dec 10, 2024
samwillis pushed a commit that referenced this pull request Dec 10, 2024
thruflo pushed a commit that referenced this pull request Dec 10, 2024
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.

2 participants