-
Notifications
You must be signed in to change notification settings - Fork 185
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
adjust performance numbers based on new benchmark results #4521
adjust performance numbers based on new benchmark results #4521
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the comma numeric separator should be changed possibly, other than that it looks good from a writing standpoint 👍
versioned_docs/version-8.6/components/best-practices/architecture/sizing-your-environment.md
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes! 🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm 👍 🚀
@akeller I'll do a docs release to get this out 👍 |
🧹 Preview environment for this PR has been torn down. |
Description
This PR updates the performance numbers based on the new results of the benchmarks (see slack message from @rodrigo-lourenco-lopes ).
These values are just valid for 8.6+.
When should this change go live?
hold
label or convert to draft PR)PR Checklist
/versioned_docs
directory (--> just valid for 8.6)./docs
directory (aka/next/
).