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
v6.2: add doc for auto analyze auto tune support #9630
v6.2: add doc for auto analyze auto tune support #9630
Conversation
[REVIEW NOTIFICATION] This pull request has been approved by:
To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by submitting an approval review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/cherry-pick master-0727 |
@TomShawn: once the present PR merges, I will cherry-pick it on top of master-0727 in the new PR and assign it to you. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the ti-community-infra/tichi repository. |
/rebase |
d8c7cf5
to
594a8da
Compare
> **Warning:** | ||
> | ||
> This feature is currently an experimental feature. It is not recommended to use this feature in the production environment. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里的改动和 quota limiter GA 无关吧, 为什么会有这个?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
顺便改的,和 feature 无关,可以忽略
tikv-configuration-file.md
Outdated
|
||
+ The soft limit on the bandwidth with which transactions write data. | ||
+ Default value: `0KB` (which means no limit) | ||
+ Recommended setting: Use the default value `0` in most cases unless the `foreground-cpu-time` setting is not enough to limit the write bandwidth. For such an exception, it is recommended to set the value smaller than 50M in the instance with 4 or less cores. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
把 50M 改成 50MB
更准确
Co-authored-by: Aolin <aolin.zhang@pingcap.com>
/remove-status LGT1 |
/merge |
This pull request has been accepted and is ready to merge. Commit hash: e3a4ddc
|
@TomShawn: new pull request created: #9856. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the ti-community-infra/tichi repository. |
What is changed, added or deleted? (Required)
v6.2: add doc for auto analyze auto tune support
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.
What is the related PR or file link(s)?
Do your changes match any of the following descriptions?