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

update default value of tidb_analyze_version in new versions #8364

Closed
wants to merge 2 commits into from
Closed

update default value of tidb_analyze_version in new versions #8364

wants to merge 2 commits into from

Conversation

yiwen92
Copy link
Contributor

@yiwen92 yiwen92 commented Feb 14, 2022 •
edited by TomShawn
Loading

First-time contributors' checklist

What is changed, added or deleted? (Required)

update default value of tidb_analyze_version in new versions

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

Tips for choosing the affected version(s):

By default, CHOOSE MASTER ONLY so your changes will be applied to the next TiDB major or minor releases. If your PR involves a product feature behavior change or a compatibility change, CHOOSE THE AFFECTED RELEASE BRANCH(ES) AND MASTER.

For details, see tips for choosing the affected versions (in Chinese).

  • master (the latest development version)
  • v5.4 (TiDB 5.4 versions)
  • v5.3 (TiDB 5.3 versions)
  • v5.2 (TiDB 5.2 versions)
  • v5.1 (TiDB 5.1 versions)
  • v5.0 (TiDB 5.0 versions)
  • 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):

Do your changes match any of the following descriptions?

  • Delete files
  • Change aliases
  • Need modification after applied to another branch
  • Might cause conflicts after applied to another branch

@ti-chi-bot
Copy link
Member

[REVIEW NOTIFICATION]

This pull request has not been approved.

To complete the pull request process, please ask the reviewers in the list to review by filling /cc @reviewer in the comment.
After your PR has acquired the required number of LGTMs, you can assign this pull request to the committer in the list by filling /assign @committer in the comment to help you merge this pull request.

The full list of commands accepted by this bot can be found here.

Reviewer can indicate their review by submitting an approval review.
Reviewer can cancel approval by submitting a request changes review.

@ti-chi-bot ti-chi-bot added missing-translation-status This PR does not have translation status info. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Feb 14, 2022
@yiwen92 yiwen92 requested a review from hfxsd February 14, 2022 10:54
@yiwen92
Copy link
Contributor Author

yiwen92 commented Feb 14, 2022

@hfxsd please review and cherry pick to target versions, thx.

@TomShawn TomShawn added area/planner Indicates that the Issue or PR belongs to the area of SQL planner or optimizer. translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. ONCALL Relates to documentation oncall. type/compatibility-or-feature-change This PR involves compatibility changes or feature behavior changes. requires-version-specific-changes After cherry-picked, the cherry-picked PR requires further changes. and removed missing-translation-status This PR does not have translation status info. labels Feb 14, 2022
@yiwen92 yiwen92 mentioned this pull request Feb 15, 2022
Merged
12 tasks
TomShawn
TomShawn reviewed Feb 17, 2022
View reviewed changes
system-variables.md
@@ -332,7 +332,7 @@ MPP 是 TiFlash 引擎提供的分布式计算框架，允许节点之间的数
- 默认值：`2`
- 范围：`[1, 2]`
- 这个变量用于控制 TiDB 收集统计信息的行为。
- 在 v5.1.0 以前的版本中，该变量的默认值为 `1`。在 v5.1.0 中，该变量的默认值为 `2`，作为实验特性使用，具体可参照[统计信息简介](/statistics.md)文档。
- 在 v5.1.0 以前的版本，该变量的值仅支持为 `1`，在 v5.1.0 版本引入变量的值 `2`。在 v5.1.0 - v5.1.3 以及 v5.2.0 - v5.2.3 中，该变量的默认值为 `2`，其作为实验特性使用（在 v5.1.4 和 v5.2.4 及其后续版本中该默认值设为 `1`）。在 v5.3.0 及其之后的新版本中，该变量的默认值设为 `2`。具体可参照[统计信息简介](/statistics.md)文档。
Copy link
Contributor

Choose a reason for hiding this comment

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

同上

@TomShawn TomShawn mentioned this pull request Feb 17, 2022
Merged
14 tasks
@TomShawn
Copy link
Contributor

本 PR 已被 #8411 所替代

@TomShawn TomShawn closed this Feb 18, 2022
@TomShawn TomShawn removed translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. requires-version-specific-changes After cherry-picked, the cherry-picked PR requires further changes. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Feb 18, 2022
@TomShawn TomShawn removed area/planner Indicates that the Issue or PR belongs to the area of SQL planner or optimizer. type/compatibility-or-feature-change This PR involves compatibility changes or feature behavior changes. ONCALL Relates to documentation oncall. labels Feb 18, 2022
@ti-chi-bot ti-chi-bot added the missing-translation-status This PR does not have translation status info. label Feb 18, 2022
@ti-chi-bot ti-chi-bot mentioned this pull request Mar 3, 2022
Merged
14 tasks
@TomShawn TomShawn mentioned this pull request Apr 11, 2022
Closed
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TomShawn TomShawn TomShawn left review comments

@winoros winoros winoros approved these changes

@hfxsd hfxsd Awaiting requested review from hfxsd

@bb7133 bb7133 Awaiting requested review from bb7133

@chrysan chrysan Awaiting requested review from chrysan

@xuyifangreeneyes xuyifangreeneyes Awaiting requested review from xuyifangreeneyes

Assignees
No one assigned
Labels
missing-translation-status This PR does not have translation status info.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@yiwen92 @ti-chi-bot @TomShawn @winoros