-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
executor,ddl: update auto_random_base in 'show create table' after insertion #18217
Conversation
…sertion Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Codecov Report
@@ Coverage Diff @@
## master #18217 +/- ##
===========================================
Coverage 79.4461% 79.4461%
===========================================
Files 535 535
Lines 144046 144046
===========================================
Hits 114439 114439
Misses 20343 20343
Partials 9264 9264 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the unclear explanation. There are three types of allocator: RowIDAllocType
, AutoIncrementType
, and AutoRandomType
.
Lines 73 to 82 in 3adb557
const ( | |
// RowIDAllocType indicates the allocator is used to allocate row id. | |
RowIDAllocType AllocatorType = iota | |
// AutoIncrementType indicates the allocator is used to allocate auto increment value. | |
AutoIncrementType | |
// AutoRandomType indicates the allocator is used to allocate auto-shard id. | |
AutoRandomType | |
// SequenceType indicates the allocator is used to allocate sequence value. | |
SequenceType | |
) |
You can construct an auto_increment
allocator for AUTO_INCREMENT = %d
clause and an auto_random
allocator for AUTO_RANDOM_BASE = %d
clause.
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
Signed-off-by: Rustin-Liu <rustin.liu@gmail.com>
@ti-srebot /run-all-tests |
/run-all-tests |
@ti-srebot /run-all-tests |
/run-all-tests |
/run-all-tests |
/run-unit-test Data Race Log
|
seems no business with this PR |
@AilinKid,Thanks for you review. |
@ti-srebot /run-all-tests |
/run-all-tests |
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
cherry pick to release-4.0 in PR #18304 |
@Rustin-Liu you can write directly to the cherry-pick pr to resolving the confilcts, which will do our a favor |
@AilinKid I will resolve it tonight. |
Signed-off-by: Rustin-Liu rustin.liu@gmail.com
What problem does this PR solve?
Issue Number: close #17456。
Problem Summary:
What is changed and how it works?
What's Changed:
Use Allocator.NextGlobalAutoID() as auto_random_base.
How it Works:
Use Allocator.NextGlobalAutoID() as auto_random_base.
Related changes
Check List
Tests
Side effects
None
Release note