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
update and clean up migration docs #18527
Conversation
Files changed:
|
✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.
|
✅ Deploy Preview for cockroachdb-api-docs canceled.
|
✅ Netlify Preview
To edit notification comments on pull requests, go to your Netlify site configuration. |
update shadowing caveats and some terminology
35f3fb8
to
8d7e724
Compare
f332c3e
to
d32e2e3
Compare
dbdd408
to
cd295b8
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.
Just left some nit comments. LGTM in general, but would like to leave the other members to finalize the review.
But IIRC from the meeting offsite, are we going to restructure the doc for LMS in future PRs, i.e. one page for testing only (shadowing) and another for production (replication and consistent cutover)?
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.
The comments I had were nits or about clarifying what you're saying. Overall lgtm
TFTRs! There is plenty of iteration left to do on these docs. In the interest of getting some important updates published, though, I'm moving this on to docs review. I can continue working on these topics in future PRs. |
Co-authored-by: Matt Linville (he/him) <linville@cockroachlabs.com>
DOC-8216
DOC-8221
DOC-9187
DOC-9192