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.

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

Merged
merged 6 commits into from
Dec 28, 2020

Conversation

sticnarf
Copy link
Contributor

@sticnarf sticnarf commented Dec 28, 2020

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)

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

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

@sticnarf sticnarf force-pushed the async-commit branch 2 times, most recently from 1a920c0 to 209c995 Compare December 28, 2020 05:22
@TomShawn
Copy link
Contributor

@sticnarf Please involve a technical review~

@TomShawn TomShawn added size/medium Changes of a medium size. status/PTAL This PR is ready for reviewing. translation/doing This PR’s assignee is translating this PR. v5.0 This PR/issue applies to TiDB v5.0 labels Dec 28, 2020
@sticnarf sticnarf force-pushed the async-commit branch 3 times, most recently from d7a242d to 0984d7a Compare December 28, 2020 05:33
Signed-off-by: Yilin Chen <sticnarf@gmail.com>
@sticnarf
Copy link
Contributor Author

PTAL @youjiali1995

Copy link
Contributor

@youjiali1995 youjiali1995 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
Copy link
Contributor

@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).

system-variables.md Outdated Show resolved Hide resolved
system-variables.md Outdated Show resolved Hide resolved
system-variables.md Outdated Show resolved Hide resolved
Copy link
Contributor

@TomShawn TomShawn 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 added the status/LGT1 Indicates that a PR has LGTM 1. label Dec 28, 2020
@TomShawn TomShawn merged commit f112d74 into pingcap:master Dec 28, 2020
> 当前该功能为实验特性,不建议在生产环境中使用。目前存在已知问题有:
>
> + 暂时与 [TiCDC](/ticdc/ticdc-overview.md) 不兼容,可能导致 TiCDC 运行不正常。
> + 暂时与 [Compaction Filter](/tikv-configuration-file.md#enable-compaction-filter) 不兼容,共同使用时有小概率发生写丢失。
Copy link
Contributor

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 时,该变量值不生效。

@sticnarf sticnarf mentioned this pull request Dec 30, 2020
10 tasks
@Joyinqin
Copy link
Contributor

/label translation/done

@ti-srebot ti-srebot added the translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. label Dec 30, 2020
@Joyinqin
Copy link
Contributor

/unlabel translation/doing

@ti-srebot ti-srebot removed the translation/doing This PR’s assignee is translating this PR. label Dec 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/medium Changes of a medium size. status/LGT1 Indicates that a PR has LGTM 1. status/PTAL This PR is ready for reviewing. translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. v5.0 This PR/issue applies to TiDB v5.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants