pdpb: support scan regions for range batch #1251
Conversation
Signed-off-by: you06 <[email protected]>
Signed-off-by: you06 <[email protected]>
Signed-off-by: you06 <[email protected]>
Signed-off-by: you06 <[email protected]>
Signed-off-by: you06 <[email protected]>
you06
changed the title
you06
changed the title
|
@JmPotato: adding LGTM is restricted to approvers and reviewers in OWNERS files. In response to this: Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
@HuSharp: adding LGTM is restricted to approvers and reviewers in OWNERS files. In response to this: Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
| @@ -53,6 +53,8 @@ service PD { | |||
|
|
|||
| rpc ScanRegions(ScanRegionsRequest) returns (ScanRegionsResponse) {} | |||
|
|
|||
| rpc BatchScanRegions(BatchScanRegionsRequest) returns (BatchScanRegionsResponse) {} | |||
There was a problem hiding this comment.
Yes, I think ScanRegions can be deprecated then.
There was a problem hiding this comment.
Then how about adding a comment for it?
Signed-off-by: you06 <[email protected]>
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cfzjywxk, HuSharp, JmPotato, rleungx The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
ref pingcap/tidb#53850
Add an interface which batch scan regions for key ranges.
The test result:
SPLIT TABLE sbtest1 BETWEEN (0) AND (1000000) REGIONS 1000;explain analyze select * from sbtest1 limit 1;...build_task_duration: 81.5ms..., statmenet execution takes 0.11 sec...build_task_duration: 2.7ms..., statement execution takes 0.02 sec