Support Custom SidecarDB Names on VTTablets#12240
Merged
mattlord merged 84 commits intovitessio:mainfrom Mar 14, 2023
Merged
Conversation
Signed-off-by: Matt Lord <mattalord@gmail.com>
Signed-off-by: Matt Lord <mattalord@gmail.com>
Contributor
Review ChecklistHello reviewers! 👋 Please follow this checklist when reviewing this Pull Request. General
If a new flag is being introduced:
If a workflow is added or modified:
Bug fixes
Non-trivial changes
New/Existing features
Backward compatibility
|
Signed-off-by: Matt Lord <mattalord@gmail.com>
Signed-off-by: Matt Lord <mattalord@gmail.com>
Signed-off-by: Matt Lord <mattalord@gmail.com>
5b61336 to
787af20
Compare
Signed-off-by: Matt Lord <mattalord@gmail.com>
787af20 to
dc1977b
Compare
Signed-off-by: Matt Lord <mattalord@gmail.com>
dd1e53c to
a9689e1
Compare
Signed-off-by: Matt Lord <mattalord@gmail.com>
a9689e1 to
0ea1f8d
Compare
5b0abb5 to
aa0cba6
Compare
Signed-off-by: Matt Lord <mattalord@gmail.com>
751f0fa to
8d17eb4
Compare
Signed-off-by: Matt Lord <mattalord@gmail.com>
8d17eb4 to
c4d6759
Compare
Signed-off-by: Matt Lord <mattalord@gmail.com>
45d4475 to
fe7e759
Compare
Signed-off-by: Matt Lord <mattalord@gmail.com>
fe7e759 to
61f7e61
Compare
Signed-off-by: Matt Lord <mattalord@gmail.com>
Signed-off-by: Matt Lord <mattalord@gmail.com>
Signed-off-by: Matt Lord <mattalord@gmail.com>
Member
Author
|
Thanks, @harshit-gangal ! I've addressed your comments here: 356b91b |
Signed-off-by: Matt Lord <mattalord@gmail.com>
Member
harshit-gangal
left a comment
There was a problem hiding this comment.
some nits, mostly looks good to me.
Signed-off-by: Matt Lord <mattalord@gmail.com>
Member
Author
Thanks for the helpful comments @harshit-gangal ! I believe that I've addressed all of them here: da4ee79 And in a follow-up: 2c11e74 |
Signed-off-by: Matt Lord <mattalord@gmail.com>
harshit-gangal
approved these changes
Mar 14, 2023
4 tasks
This was referenced May 8, 2023
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR adds a new experimental feature: keyspace level sidecar database names. For the why, please see the feature request.
In this PR the sidecar db name can only be set when the keyspace is first created via the
CreateKeyspacevtctl client command using the new--sidecar-db-namecommand flag. From that point forward it is immutable (w/o modifying the topo directly outside of Vitess). If no value is specified for the keyspace then it will be set to the default of_vtwhen the keyspace is created, and as a fallback the default will also be set when the first tablet is started within the keyspace if for any reason it's not already set in the topo keyspace record.The management of the sidecar database resides entirely within a
vttablet. This allows eachvttabletin a keyspace to manage its own sidecar database independently of all other keyspaces and processes in the cluster. This allows e.g. for most tablets within a Vitess cluster to use the default sidecar name of_vtwhile some can use a non-standard name of e.g._vt_importor__virtual_vitess_clusterfor whatever purposes as needed (in various cases the tablet'smysqldinstance may be participating in two distinct vitess clusters or two distinct keyspaces). From outside of thevttablet, Vitess is able to assume that the standard name of_vtis being used (including external/3rd party tooling that e.g. may be doing things likevtctlclient ExecuteFetchAsDba) unless talking directly to a tablet's mysqld instance.Related Issue(s)
Checklist