Skip to content

update SPM document - #2679

Merged
sre-bot merged 4 commits into
pingcap:masterfrom
TomShawn:SPM-doc
Jun 8, 2020
Merged

update SPM document#2679
sre-bot merged 4 commits into
pingcap:masterfrom
TomShawn:SPM-doc

Conversation

@TomShawn

@TomShawn TomShawn commented Jun 4, 2020

Copy link
Copy Markdown
Contributor

What is changed, added or deleted? (Required)

Mention the SQL plan management in sql-binding.md。

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)

If you select two or more versions from above, to trigger the bot to cherry-pick this PR to your desired release version branch(es), you must add corresponding labels such as needs-cherry-pick-4.0, needs-cherry-pick-3.1, needs-cherry-pick-3.0, and needs-cherry-pick-2.1.

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

@TomShawn TomShawn added translation/from-docs-cn This PR is translated from a PR in pingcap/docs-cn. status/PTAL This PR is ready for reviewing. needs-cherry-pick-4.0 labels Jun 4, 2020
@TomShawn
TomShawn requested review from lzmhhh123 and yikeke June 4, 2020 09:25
@sre-bot

sre-bot commented Jun 6, 2020

Copy link
Copy Markdown
Contributor

@lzmhhh123, @yikeke, PTAL.

@yikeke
yikeke requested a review from CaitinChen June 8, 2020 05:10
@yikeke

yikeke commented Jun 8, 2020

Copy link
Copy Markdown
Contributor

Please help review this PR, thanks~ @CaitinChen

@yikeke
yikeke removed their request for review June 8, 2020 05:15

@lzmhhh123 lzmhhh123 left a comment

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.

LGTM

@lzmhhh123 lzmhhh123 added the status/LGT1 Indicates that a PR has LGTM 1. label Jun 8, 2020
Comment thread sql-plan-management.md Outdated
Comment thread sql-plan-management.md Outdated
Comment thread sql-plan-management.md Outdated
Comment thread sql-plan-management.md Outdated
Comment thread sql-plan-management.md Outdated
Comment thread sql-plan-management.md Outdated
Comment thread sql-plan-management.md Outdated

> **Note:**
>
> Because TiDB has some embedded SQL statements to ensure the correctness of some functions, baseline capturing by default automatically shields these SQL statements.

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.

"the correctness of some functions" or "the correctness of some features"?
If we use "functions," users might confuse it with 函数.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Changed to feature.

Comment thread sql-plan-management.md Outdated
Co-authored-by: Caitin <34535727+CaitinChen@users.noreply.github.com>
Comment thread sql-plan-management.md Outdated
@TomShawn

TomShawn commented Jun 8, 2020

Copy link
Copy Markdown
Contributor Author

@CaitinChen All comments addressed, PTAL again, thanks!

@CaitinChen CaitinChen left a comment

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.

LGTM

@TomShawn TomShawn added status/can-merge Indicates a PR has been approved by a committer. and removed status/LGT1 Indicates that a PR has LGTM 1. status/PTAL This PR is ready for reviewing. labels Jun 8, 2020
@sre-bot

sre-bot commented Jun 8, 2020

Copy link
Copy Markdown
Contributor

/run-all-tests

@sre-bot
sre-bot merged commit 71e8033 into pingcap:master Jun 8, 2020
sre-bot pushed a commit to sre-bot/docs that referenced this pull request Jun 8, 2020
Signed-off-by: sre-bot <sre-bot@pingcap.com>
@sre-bot sre-bot mentioned this pull request Jun 8, 2020
5 tasks
@sre-bot

sre-bot commented Jun 8, 2020

Copy link
Copy Markdown
Contributor

cherry pick to release-4.0 in PR #2736

@TomShawn
TomShawn deleted the SPM-doc branch June 8, 2020 12:01
sre-bot added a commit that referenced this pull request Jun 8, 2020
Signed-off-by: sre-bot <sre-bot@pingcap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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