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

docs: new RFC on tenant routing #93537

Merged
merged 1 commit into from
Dec 21, 2022
Merged

Conversation

knz
Copy link
Contributor

@knz knz commented Dec 13, 2022

Rendered RFC text here

Informs #92619.
Informs #92526.
Informs #92580.

Epic: CRDB-22385
Release note: None

@knz knz requested a review from a team as a code owner December 13, 2022 19:34
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@knz knz force-pushed the 20221212-tenant-routing branch from 7dfa9ab to a25827a Compare December 13, 2022 19:35
@knz knz added A-multitenancy Related to multi-tenancy T-multitenant Issues owned by the multi-tenant virtual team labels Dec 13, 2022
@knz knz requested a review from stevendanna December 13, 2022 19:38
Epic: CRDB-22385
Release note: None
Copy link
Collaborator

@jeffswenson jeffswenson left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you for writing this! I like the cluster:tenant-32/database format for embedding routing information in the database name.

@knz
Copy link
Contributor Author

knz commented Dec 21, 2022

TFYR!

bors r=JeffSwenson,andy-kimball

@craig
Copy link
Contributor

craig bot commented Dec 21, 2022

Build succeeded:

@craig craig bot merged commit 7d47c0f into cockroachdb:master Dec 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-multitenancy Related to multi-tenancy T-multitenant Issues owned by the multi-tenant virtual team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants