Skip to content
This repository has been archived by the owner on Jun 24, 2021. It is now read-only.

scripts: update the KV metrics on TiDB #1080

Merged
merged 2 commits into from
Dec 24, 2019

Conversation

jackysp
Copy link
Member

@jackysp jackysp commented Dec 23, 2019

Signed-off-by: Shuaipeng Yu [email protected]
Avoid using tidb_tikvclient_txn_cmd_total for KV request which will confuse the user.
企业微信截图_20191223165008
企业微信截图_20191223165024

Copy link

@imtbkcat imtbkcat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@imtbkcat imtbkcat added the LGT1 label Dec 24, 2019
@@ -5688,18 +5688,18 @@
"dashLength": 10,
"dashes": false,
"datasource": "${DS_TEST-CLUSTER}",
"description": "kv requests durations by store",
"description": "kv command total by instance and command type",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be kv requests total?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

addressed

Signed-off-by: Shuaipeng Yu <[email protected]>
@coocood
Copy link
Member

coocood commented Dec 24, 2019

LGTM

@jackysp jackysp merged commit 2003174 into pingcap:master Dec 24, 2019
@jackysp jackysp deleted the update_kv_metrics branch March 18, 2020 06:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants