Skip to content

Document VReplication Source Throttling For Copy Table#1002

Merged
mattlord merged 3 commits intoprodfrom
PR_vitessio/9923
Apr 5, 2022
Merged

Document VReplication Source Throttling For Copy Table#1002
mattlord merged 3 commits intoprodfrom
PR_vitessio/9923

Conversation

@mattlord
Copy link
Copy Markdown
Member

@mattlord mattlord commented Mar 31, 2022

This documents the work done in vitessio/vitess#9923

I also used this one liner to move all long flags from single dash to double dash (single is deprecated in 14.0) in the files edited in the PR:

sed -e ':duit' -r -i '' -e 's/(.*[[:space:]]+[`]*)(-[a-z,A-Z]+[a-z,A-Z,-_`]+)([[:space:]]+.*)/\1-\2\3/g' -e 't duit' <path_to_file>

That gets most of them and a manual review can be done to ensure everything is good.

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

netlify bot commented Mar 31, 2022

Deploy Preview for vitess ready!

Name Link
🔨 Latest commit 5397c1d
🔍 Latest deploy log https://app.netlify.com/sites/vitess/deploys/624c6cb183e9010008c3ecaf
😎 Deploy Preview https://deploy-preview-1002--vitess.netlify.app/docs/14.0/reference/vreplication/flags
📱 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 force-pushed the PR_vitessio/9923 branch 6 times, most recently from 740f5b7 to 5aa5c27 Compare March 31, 2022 15:59
@mattlord mattlord marked this pull request as ready for review March 31, 2022 16:02
@mattlord mattlord requested a review from rohit-nayak-ps March 31, 2022 16:02
@mattlord mattlord force-pushed the PR_vitessio/9923 branch 4 times, most recently from 62b4a06 to 2a2d9c7 Compare March 31, 2022 21:21
@mattlord mattlord removed the request for review from rohit-nayak-ps April 5, 2022 14:45
Copy link
Copy Markdown
Collaborator

@FancyFane FancyFane left a comment

Choose a reason for hiding this comment

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

Reviewing the changes I only had one question for the technical content.

Signed-off-by: Matt Lord <mattalord@gmail.com>
@mattlord mattlord requested a review from FancyFane April 5, 2022 16:24
Copy link
Copy Markdown
Collaborator

@FancyFane FancyFane left a comment

Choose a reason for hiding this comment

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

lgtm

@mattlord mattlord merged commit e32e24d into prod Apr 5, 2022
@mattlord mattlord deleted the PR_vitessio/9923 branch April 5, 2022 17:06
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