Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

reference: add documentation for statement summary #1939

Merged
merged 3 commits into from
Oct 14, 2019

Conversation

djshow832
Copy link
Contributor

@djshow832 djshow832 commented Oct 12, 2019

What is changed, added or deleted?

Add documentation for statement summary.

Statement summary is recently released, and a documentation is needed to guide users.

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

#1923 (closed)
master: pingcap/tidb#12017, pingcap/tidb#12217
release 3.0: pingcap/tidb#12306, pingcap/tidb#12308
release 3.1: pingcap/tidb#12328, pingcap/tidb#12499

Which version does your change affect?

dev, v3.0

@djshow832 djshow832 added v3.0 translation/doing This PR’s assignee is translating this PR. labels Oct 12, 2019
@djshow832 djshow832 requested review from bb7133 and dcalvin October 12, 2019 09:12

## 排查示例

下面来两个示例问题演示如何利用 statement summary 来排查。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
下面来两个示例问题演示如何利用 statement summary 来排查。
下面用两个示例问题演示如何利用 statement summary 来排查。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Addressed.

@dcalvin dcalvin added translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. and removed translation/doing This PR’s assignee is translating this PR. labels Oct 12, 2019
Copy link
Member

@bb7133 bb7133 left a comment

Choose a reason for hiding this comment

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

LGTM

@bb7133 bb7133 added the status/LGT1 Indicates that a PR has LGTM 1. label Oct 14, 2019
Copy link
Contributor

@dcalvin dcalvin left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/LGT1 Indicates that a PR has LGTM 1. translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants