Conversation
Signed-off-by: qidi1 <1083369179@qq.com>
|
[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. DetailsReviewer can indicate their review by submitting an approval review. |
|
|
|
This is very cool! Will you also propose the CPU dimension in this PR, or in later PRs? |
Codecov Report
@@ Coverage Diff @@
## master #1252 +/- ##
==========================================
+ Coverage 31.53% 31.87% +0.34%
==========================================
Files 290 297 +7
Lines 17097 17118 +21
Branches 683 680 -3
==========================================
+ Hits 5391 5456 +65
+ Misses 11475 11431 -44
Partials 231 231
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
|
@qidi1: PR needs rebase. DetailsInstructions 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. |
Signed-off-by: qidi1 1083369179@qq.com
What problem does this PR solve?
Currently,Keyviz has
keysandbytesdimensions. In some situations, it cannot find the hotspot that the CPU is high caused by QPS, thekeysandbytesmay be 0. For example:Issue Number: #1209
What is changed and how does it work?
Return QPS data in PD.
Check List
Tests
Related changes
Release note
Manual test
We ran a small cluster locally. On the cluster, test data was generated using the tiup bench tool. The results are shown in the figure below.

