-
Notifications
You must be signed in to change notification settings - Fork 411
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
support keyspace feature #7039
support keyspace feature #7039
Conversation
Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: yongman <[email protected]>
Signed-off-by: yongman <[email protected]>
Signed-off-by: yongman <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
1. refine some log messages, add a child logger with keyspace id info 2. add some comments for ManagedStorages's interfaces Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: JaySon-Huang <[email protected]>
Signed-off-by: JaySon-Huang <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
1. move api_version for TiFlashStorageConfig 2. refine comments for TMTStorages Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
…ple config Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
[REVIEW NOTIFICATION] This pull request has not been approved. To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by submitting an approval review. |
Signed-off-by: iosmanthus [email protected]
What problem does this PR solve?
Issue Number: close #6815
Original pull request: #6816
Problem Summary:
Deps:
What is changed and how it works?
This pull request separates each keyspace's db/tables by rewriting their name with a keyspace prefix.
For example, table 1 in keyspace 2 is stored in a directory named:
ks_2_t_1
, and the same as the schema SQL file.The keyspace's information is decoded from the region range in the writing process, and from gRPC requests context in the reading process. Every time TiFlash
syncSchema
, it needs a keyspace id parameter. The persistent schema will also contain the keyspace id in case of restarting the server.This pull request also introduces a new config named
api-version
, whose default value is 1 meaning the node is working in the legacy mode. In this mode, schema info will be fetched in the bootstrap stage. If theapi-version
is 2, then the node is working in the keyspace mode, which will fetch schema lazily.The pull request does not support the PageStoreV3 format and needs to be tested in V2 for now. In the future, the pull request should be compatible with the
UniversalPageStore
feature.Check List
Tests
Side effects
Documentation
Release note