raftstore, tikv_util: move QueryStats from raftstore to tikv_util component #13506
Conversation
Signed-off-by: zeminzhou <zhouzemin@pingcap.com>
|
[REVIEW NOTIFICATION] This pull request has been approved by:
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: zeminzhou <zhouzemin@pingcap.com>
|
/cc @pingyu @haojinming |
Signed-off-by: zeminzhou <zhouzemin@pingcap.com>
Signed-off-by: zeminzhou <zhouzemin@pingcap.com>
Signed-off-by: zeminzhou <zhouzemin@pingcap.com>
| @@ -0,0 +1,14 @@ | |||
| // Copyright 2016 TiKV Project Authors. Licensed under Apache-2.0. | |||
There was a problem hiding this comment.
2016 -> 2022. The same to other new files.
There was a problem hiding this comment.
Done, thinks, PTAL~
| .iter() | ||
| .any(|rp| rp.get_store_id() == lp.get_store_id() && rp.get_role() == lp.get_role()) | ||
| }) | ||
| } |
There was a problem hiding this comment.
Suggest to move unit tests from raftstore to here.
Signed-off-by: zeminzhou <zhouzemin@pingcap.com>
Signed-off-by: zeminzhou <zhouzemin@pingcap.com>
ti-chi-bot
added
status/LGT2
|
/cc @Connor1996 PTAL~, thanks! |
|
/merge |
|
@Connor1996: It seems you want to merge this PR, I will help you trigger all the tests: /run-all-tests You only need to trigger If you have any questions about the PR merge process, please refer to pr process. 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 ti-community-infra/tichi repository. |
|
This pull request has been accepted and is ready to merge. Commit hash: e3ba3bb
|
Signed-off-by: zeminzhou zhouzemin@pingcap.com
What is changed and how it works?
Issue Number: close#13505
What's Changed:
Check List
Tests