Skip to content

Remove documentation related to legacy sharding in 15.0+#1090

Merged
mattlord merged 2 commits intoprodfrom
delete_legacy_sharding
Jul 12, 2022
Merged

Remove documentation related to legacy sharding in 15.0+#1090
mattlord merged 2 commits intoprodfrom
delete_legacy_sharding

Conversation

@mattlord
Copy link
Copy Markdown
Member

@mattlord mattlord commented Jul 7, 2022

This documents: vitessio/vitess#10278

Signed-off-by: Matt Lord <mattalord@gmail.com>
@netlify
Copy link
Copy Markdown

netlify bot commented Jul 7, 2022

Deploy Preview for vitess ready!

Name Link
🔨 Latest commit fa670db
🔍 Latest deploy log https://app.netlify.com/sites/vitess/deploys/62cdbb5ab5af8b0009499bc6
😎 Deploy Preview https://deploy-preview-1090--vitess.netlify.app/docs/15.0/reference/programs/vtctl/shards
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@mattlord mattlord requested review from ajm188 and deepthi July 7, 2022 17:04
@mattlord mattlord changed the title Remove documentation related to legacy sharding Remove documentation related to legacy sharding in 15.0+ Jul 7, 2022
Copy link
Copy Markdown
Collaborator

@deepthi deepthi left a comment

Choose a reason for hiding this comment

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

Mostly LGTM.
Is there anything left under "historical" in User Guides? If not, we can delete that directory too.

@mattlord
Copy link
Copy Markdown
Member Author

Is there anything left under "historical" in User Guides? If not, we can delete that directory too.

The directory is gone, although that's not clear in the PR.

Do we really want to keep the "vertical split" wording?

No, good catch! I'll address that before merging. Thank you! ❤️

Signed-off-by: Matt Lord <mattalord@gmail.com>
@mattlord mattlord merged commit 4b8ba52 into prod Jul 12, 2022
@mattlord mattlord deleted the delete_legacy_sharding branch July 12, 2022 18:26
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