Skip to content

*: update grafana-tidb-dashboard#3067

Merged
ti-srebot merged 5 commits into
pingcap:masterfrom
zimulala:grafana
Jul 14, 2020
Merged

*: update grafana-tidb-dashboard#3067
ti-srebot merged 5 commits into
pingcap:masterfrom
zimulala:grafana

Conversation

@zimulala
Copy link
Copy Markdown
Contributor

@zimulala zimulala commented Jun 30, 2020

What is changed, added or deleted? (Required)

Update TiDB metrics

Which TiDB version(s) do your changes apply to? (Required)

  • master (the latest development version)
  • v4.0 (TiDB 4.0 versions)
  • v3.1 (TiDB 3.1 versions)
  • v3.0 (TiDB 3.0 versions)
  • v2.1 (TiDB 2.1 versions)

What is the related PR or file link(s)?

@zimulala zimulala added status/PTAL This PR is ready for reviewing. and removed status/PTAL This PR is ready for reviewing. labels Jun 30, 2020
@yikeke yikeke requested review from jackysp and lilin90 June 30, 2020 03:24
@yikeke yikeke added status/PTAL This PR is ready for reviewing. translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn. labels Jun 30, 2020
@yikeke yikeke requested review from TomShawn and removed request for lilin90 June 30, 2020 03:31
Copy link
Copy Markdown
Contributor

@jackysp jackysp left a comment

Choose a reason for hiding this comment

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

LGTM, but not sure about the 'words'.

@TomShawn TomShawn added the special-week PR from Document Special Week. label Jun 30, 2020
@ti-srebot
Copy link
Copy Markdown
Contributor

@jackysp, @TomShawn, PTAL.

1 similar comment
@ti-srebot
Copy link
Copy Markdown
Contributor

@jackysp, @TomShawn, PTAL.

Copy link
Copy Markdown
Contributor

@TomShawn TomShawn left a comment

Choose a reason for hiding this comment

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

The first round of review
I'll review the rest part soon. @zimulala You can address these comments first.

Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Copy link
Copy Markdown
Contributor

@TomShawn TomShawn left a comment

Choose a reason for hiding this comment

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

The second round of review

Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated

- Owner
- New ETCD Session Duration 95: the time it takes to create a new etcd session. TiDB connects to etcd in PD through etcd client to save/read some metadata information. This records the time spent creating the session
- Owner Watcher OPS: the number of operations per second of DDL owner watches PD's etcd metadata
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@jackysp The Chinese version DDL owner watch PD 的 etcd 的元数据的 goroutine 的每秒操作次数 is very confusing. Please help check whether the translation here is correct, thanks!

Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
@ti-srebot
Copy link
Copy Markdown
Contributor

@jackysp, @TomShawn, PTAL.

1 similar comment
@ti-srebot
Copy link
Copy Markdown
Contributor

@jackysp, @TomShawn, PTAL.

Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
Comment thread grafana-tidb-dashboard.md Outdated
@TomShawn
Copy link
Copy Markdown
Contributor

LGTM
@jackysp PTAL

Copy link
Copy Markdown
Contributor

@jackysp jackysp left a comment

Choose a reason for hiding this comment

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

lgtm

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Jul 14, 2020
Comment thread grafana-tidb-dashboard.md Outdated
@TomShawn
Copy link
Copy Markdown
Contributor

/merge

@ti-srebot ti-srebot added the status/can-merge Indicates a PR has been approved by a committer. label Jul 14, 2020
@ti-srebot
Copy link
Copy Markdown
Contributor

/run-all-tests

@ti-srebot ti-srebot merged commit d67776e into pingcap:master Jul 14, 2020
@TomShawn TomShawn removed status/LGT1 Indicates that a PR has LGTM 1. status/PTAL This PR is ready for reviewing. labels Jul 14, 2020
@TomShawn
Copy link
Copy Markdown
Contributor

/run-cherry-picker

ti-srebot pushed a commit to ti-srebot/docs that referenced this pull request Jul 14, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot
Copy link
Copy Markdown
Contributor

cherry pick to release-4.0 in PR #3265

@zimulala zimulala deleted the grafana branch July 14, 2020 05:56
ti-srebot added a commit that referenced this pull request Jul 14, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

special-week PR from Document Special Week. status/can-merge Indicates a PR has been approved by a committer. translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants