-
Notifications
You must be signed in to change notification settings - Fork 70
Blog post: Introducing Atomic Slot Migration #404
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
Conversation
Signed-off-by: Jacob Murphy <jkmurphy@google.com>
Signed-off-by: Kyle J. Davis <kyledvs@amazon.com>
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 really like this blog post. I had quite a few very minor edits and questions, but the overall blog post is very good. Additionally, I made a few edits directly on your fork as a PR:
Signed-off-by: Jacob Murphy <jkmurphy@google.com>
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
|
@madolson let me know if you'd like a look. Also not sure what date to put for the release, I guess I will just update it to the current day before merge. Not sure if there is another planned for this week and we should space it out |
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.
Just did a tech review, and nothing seems wrong or misleading to me, so I'm good.
Description
Adds a blog post for the Atomic Slot Migration feature.
Issues Resolved
closes #369
Check List
--signoffBy submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.