-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
update async commit related configurations #5181
Conversation
1a920c0
to
209c995
Compare
@sticnarf Please involve a technical review~ |
d7a242d
to
0984d7a
Compare
Signed-off-by: Yilin Chen <[email protected]>
0984d7a
to
e6d19e6
Compare
PTAL @youjiali1995 |
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
@youjiali1995, Thanks for your review. The bot only counts LGTMs from Reviewers and higher roles, but you're still welcome to leave your comments. See the corresponding SIG page for more information. Related SIG: docs(slack). |
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
> 当前该功能为实验特性,不建议在生产环境中使用。目前存在已知问题有: | ||
> | ||
> + 暂时与 [TiCDC](/ticdc/ticdc-overview.md) 不兼容,可能导致 TiCDC 运行不正常。 | ||
> + 暂时与 [Compaction Filter](/tikv-configuration-file.md#enable-compaction-filter) 不兼容,共同使用时有小概率发生写丢失。 |
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.
当前该功能为实验特性,不建议在生产环境中使用。目前存在已知问题有
修改成:
实验特性,不推荐在生产环境中使用。目前已知不兼容性,如下:
本特性与 TiDB Binlog 不兼容,开启 TiDB Binlog 时本配置将不生效
修改成:开启 TiDB Binlog 时,该变量值不生效。
/label translation/done |
/unlabel translation/doing |
First-time contributors' checklist
What is changed, added or deleted? (Required)
Async commit and 1PC configurations have been moved from config file to system variables.
And there are known correctness issues about 1PC, so 1PC content is hidden.
Which TiDB version(s) do your changes apply to? (Required)
Do your changes match any of the following descriptions?