Skip to content
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

Add flag to keep routing rules in v2 vrepl workflows #9441

Merged
merged 2 commits into from
Dec 27, 2021

Conversation

mattlord
Copy link
Contributor

@mattlord mattlord commented Dec 24, 2021

Description

Once you've completed a MoveTables e.g., you may want to keep the routing rules around for a while as you find and fix any application code that is still using the old keyspace.

This adds a -keep_routing_rules flag to the vtctl MoveTables {Complete,Cancel} commands for the v2 workflows to support this use case.

Related Issue(s)

Checklist

Deployment Notes

Signed-off-by: Matt Lord <mattalord@gmail.com>
@mattlord mattlord added Component: VReplication Type: Enhancement Logical improvement (somewhere between a bug and feature) release notes labels Dec 24, 2021
@mattlord mattlord force-pushed the vrepl_complete_keep_rr branch 3 times, most recently from c6ec959 to 540b4ae Compare December 26, 2021 03:23
@mattlord mattlord changed the title Add option to keep routing rules in v2 vrepl workflows Add flag to keep routing rules in v2 vrepl workflows Dec 26, 2021
@mattlord mattlord marked this pull request as ready for review December 26, 2021 05:08
Signed-off-by: Matt Lord <mattalord@gmail.com>
@mattlord mattlord force-pushed the vrepl_complete_keep_rr branch from 540b4ae to 90ad0a4 Compare December 26, 2021 19:51
Copy link
Contributor

@rohit-nayak-ps rohit-nayak-ps left a comment

Choose a reason for hiding this comment

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

lgtm

@rohit-nayak-ps rohit-nayak-ps merged commit d36cd5e into vitessio:main Dec 27, 2021
@rohit-nayak-ps rohit-nayak-ps deleted the vrepl_complete_keep_rr branch December 27, 2021 11:17
mattlord added a commit to vitessio/website that referenced this pull request Dec 27, 2021
Added in: vitessio/vitess#9441

Signed-off-by: Matt Lord <mattalord@gmail.com>
mattlord added a commit to vitessio/website that referenced this pull request Dec 27, 2021
Added in: vitessio/vitess#9441

Signed-off-by: Matt Lord <mattalord@gmail.com>
mattlord added a commit to vitessio/website that referenced this pull request Dec 27, 2021
Added in: vitessio/vitess#9441

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

jfg956 commented Nov 10, 2023

Revisiting this, and thought to add below as it might be a relevant discussion.
https://vitess.slack.com/archives/C0PQY0PTK/p1640367070469000

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: VReplication Type: Enhancement Logical improvement (somewhere between a bug and feature)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants