-
Notifications
You must be signed in to change notification settings - Fork 296
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
docs: Add release notes for v1.7.0. #2858
Conversation
full administrative priveleges. -> privileges. |
ee51690
to
59e102f
Compare
59e102f
to
9b3db0f
Compare
9b3db0f
to
5d7342b
Compare
5d7342b
to
cfe9406
Compare
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.
Looks great to me. It must have taken quite some time to go through all those PRs.
Since there are so many performance enhancements in this release, I ran some initial sync tests to see the overall relative difference in performance between release-v1.6
and release-v1.7
. Based on my testing:
release-v1.7
is ~47.87% faster thanrelease-v1.6
with default settingsrelease-v1.7
is ~39.36% faster thanrelease-v1.6
when using the latest checkpoint from 1.6 (this is to compare without givingrelease-v1.7
the advantage of a more recent checkpoint)release-v1.7
is ~29.37% faster thanrelease-v1.6
with checkpoints disabled
Details of the tests I ran are in this gist.
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.
Reads well. Only thing I noticed which could be improved is punctuation for list items with sentences, a few are punctuated while majority are not.
Thanks for those numbers @rstaudt2. I added a section entitled I went back and forth a bit on whether to include the others, but I think it's probably better to keep it more high level to strike a balance. I also updated the |
cfe9406
to
e4b783f
Compare
Nice, that looks good to me. Agreed that the first value makes the most sense to put there. |
e4b783f
to
745c497
Compare
f3b65b7
to
48b16cf
Compare
df28ffb
to
40dd5d1
Compare
40dd5d1
to
819805f
Compare
This is currently the release notes for rc1 for the purposes of review. It will be updated to remove the RC status and merged once the final non-rc release is ready.