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.

Sign up for GitHub

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

Jump to bottom

TiDB: add a new document about how to analyze slow queries #3858

Merged
merged 45 commits into from
Sep 28, 2020
Merged

TiDB: add a new document about how to analyze slow queries #3858

merged 45 commits into from
Sep 28, 2020

Conversation

qw4990
Copy link
Contributor

@qw4990 qw4990 commented Jul 2, 2020 •
edited
Loading

What is changed, added or deleted? (Required)

Add a new document about how to analyze slow queries.

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)?

  • This PR is translated from:
  • Other reference link(s):

@yikeke
Copy link
Contributor

yikeke commented Jul 2, 2020

Do your changes apply to v4.0? @qw4990

@yikeke yikeke added status/PTAL This PR is ready for reviewing. translation/welcome Waits for a contributor to translate this PR and create a PR to the pingcap/docs-cn repository. labels Jul 2, 2020
@qw4990
Copy link
Contributor Author

qw4990 commented Jul 2, 2020

Do your changes apply to v4.0? @qw4990

Yes. The description is updated now.

@qw4990 qw4990 requested review from SunRunAway and superlzs0476 July 2, 2020 07:57
zz-jason
zz-jason reviewed Jul 2, 2020
View reviewed changes
breezewish
breezewish reviewed Jul 2, 2020
View reviewed changes
@TomShawn TomShawn added needs-cherry-pick-4.0 size/large Changes of a large size. labels Jul 3, 2020
@ti-srebot
Copy link
Contributor

@zz-jason, @breeswish, @SunRunAway, @winoros, @yikeke, @superlzs0476, PTAL.

1 similar comment
@ti-srebot
Copy link
Contributor

@zz-jason, @breeswish, @SunRunAway, @winoros, @yikeke, @superlzs0476, PTAL.

AndreMouche
AndreMouche reviewed Jul 6, 2020
View reviewed changes
Copy link
Contributor

@AndreMouche AndreMouche left a comment

Choose a reason for hiding this comment

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

@yikeke PTAL thanks

@yikeke
Copy link
Contributor

yikeke commented Jul 6, 2020

A lot of blanks-around-xxx markdown lint errors are reported in CI. Please fix them first. @qw4990

@yikeke yikeke added the status/require-change Needs the author to address comments. label Jul 6, 2020
yikeke
yikeke reviewed Jul 6, 2020
View reviewed changes
@ti-srebot
Copy link
Contributor

@zz-jason, @breeswish, @AndreMouche, @yikeke, @SunRunAway, @winoros, @superlzs0476, PTAL.

@ti-srebot
Copy link
Contributor

@qw4990, please update your pull request.

@ti-srebot
Copy link
Contributor

@zz-jason, @breeswish, @AndreMouche, @yikeke, @SunRunAway, @winoros, @superlzs0476, PTAL.

@Joyinqin Joyinqin mentioned this pull request Sep 23, 2020
Merged
9 tasks
@ti-srebot
Copy link
Contributor

@qw4990, please update your pull request.

@ti-srebot
Copy link
Contributor

@zz-jason, @breeswish, @AndreMouche, @yikeke, @SunRunAway, @winoros, @superlzs0476, PTAL.

@ti-srebot
Copy link
Contributor

@qw4990, please update your pull request.

@ti-srebot
Copy link
Contributor

@zz-jason, @breeswish, @AndreMouche, @yikeke, @SunRunAway, @winoros, @superlzs0476, PTAL.

SunRunAway
SunRunAway reviewed Sep 28, 2020
View reviewed changes
Copy link
Contributor

@SunRunAway SunRunAway 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 removed the status/LGT1 Indicates that a PR has LGTM 1. label Sep 28, 2020
@ti-srebot ti-srebot added the status/LGT2 Indicates that a PR has LGTM 2. label Sep 28, 2020
yikeke
yikeke approved these changes Sep 28, 2020
View reviewed changes
Copy link
Contributor

@yikeke yikeke left a comment

Choose a reason for hiding this comment

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

LGTM

@yikeke yikeke merged commit af8ead0 into pingcap:master Sep 28, 2020
ti-srebot pushed a commit to ti-srebot/docs-cn that referenced this pull request Sep 28, 2020
@qw4990 @ti-srebot
cherry pick pingcap#3858 to release-4.0
b1da2f6 
Signed-off-by: ti-srebot <[email protected]>
@ti-srebot ti-srebot mentioned this pull request Sep 28, 2020
Merged
5 tasks
@ti-srebot
Copy link
Contributor

cherry pick to release-4.0 in PR #4615

yikeke pushed a commit that referenced this pull request Sep 28, 2020
@ti-srebot @qw4990
cherry pick #3858 to release-4.0 (#4615)
e220de7 
Signed-off-by: ti-srebot <[email protected]>

Co-authored-by: Yuanjia Zhang <[email protected]>
@ti-srebot ti-srebot mentioned this pull request Oct 12, 2020
Merged
9 tasks
@TomShawn TomShawn mentioned this pull request Oct 12, 2020
Merged
9 tasks
@TomShawn TomShawn added translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. and removed status/require-change Needs the author to address comments. translation/doing This PR’s assignee is translating this PR. labels Oct 13, 2020
This was referenced Oct 15, 2020
Merged
Merged
This was referenced Oct 26, 2020
Merged
Merged
@TomShawn TomShawn mentioned this pull request Nov 16, 2020
Merged
9 tasks
@ti-srebot ti-srebot mentioned this pull request Nov 20, 2020
Merged
9 tasks
This was referenced Feb 20, 2021
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SunRunAway SunRunAway SunRunAway left review comments

@AndreMouche AndreMouche AndreMouche left review comments

@breezewish breezewish breezewish left review comments

@zz-jason zz-jason zz-jason left review comments

@yikeke yikeke yikeke approved these changes

@ti-srebot ti-srebot ti-srebot approved these changes

@winoros winoros Awaiting requested review from winoros

@superlzs0476 superlzs0476 Awaiting requested review from superlzs0476

Assignees

@TomShawn TomShawn

Labels
size/large Changes of a large size. status/LGT2 Indicates that a PR has LGTM 2. translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@qw4990 @yikeke @ti-srebot @SunRunAway @cfzjywxk @AndreMouche @breezewish @zz-jason @TomShawn