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

reference/configuration: add description about TiDB txn-total-size-limit #2456

Merged
merged 6 commits into from
Apr 8, 2020

Conversation

tiancaiamao
Copy link
Contributor

@tiancaiamao tiancaiamao added size/large Changes of a large size. v4.0 This PR/issue applies to TiDB v4.0. size/small Changes of a small size. and removed size/large Changes of a large size. labels Mar 13, 2020
@tiancaiamao
Copy link
Contributor Author

PTAL @jackysp @TomShawn

@TomShawn TomShawn added the translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. label Mar 13, 2020

+ TiDB 事务大小限制
+ 默认值:104857600
+ 单个事务中,所有的 key-value 记录的总大小不能超过该限制。注意,如果 `binlog` 开启了,该配置项的值不能超过 104857600(100M),因为 `binlog` 组件不支持同步的事务过大;如果 `binlog` 没开启,该配置项的最大值不超过 10737418240(10G)。
Copy link
Member

Choose a reason for hiding this comment

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

This comment may not be related to the documentation. Should we set a lower limit, if the size is too low, TiDB will not start.

@TomShawn
Copy link
Contributor

@tiancaiamao Please address the above comments, thanks!

@tiancaiamao
Copy link
Contributor Author

PTAL @TomShawn @jackysp

@@ -209,3 +209,5 @@ COMMIT;
为了使性能达到最优,建议每 100~500 行写入一个事务。

TiDB 设置了单个事务的键值对的总大小不超过 100 MB,这个默认值可以通过配置文件中的配置项 `txn-total-size-limit` 进行修改,最大支持到 10GB。实际的单个事务大小限制还取决于用户的内存,执行大事务时 TiDB 进程的内存消耗大约是事务大小 6 倍以上。

在 4.0 以前的版本,TiDB 限制了单个事务的键值对的总数量不超过 30 万条,在 4.0 之后的版本没有这项限制。
Copy link
Contributor

Choose a reason for hiding this comment

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

Please update this part in #2002.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The whole block is missing

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

@tiancaiamao The missing part is aligned in pingcap/docs#2280. PTAL

Copy link
Member

@jackysp jackysp left a comment

Choose a reason for hiding this comment

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

LGTM

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

@TomShawn TomShawn added the status/can-merge Indicates a PR has been approved by a committer. label Apr 8, 2020
@sre-bot
Copy link
Contributor

sre-bot commented Apr 8, 2020

/run-all-tests

@TomShawn TomShawn added the requires-followup This PR requires a follow-up task after being merged. label Apr 8, 2020
@sre-bot sre-bot merged commit 19cce20 into pingcap:master Apr 8, 2020
yikeke added a commit that referenced this pull request Apr 8, 2020
)

* *: update some guiding resources #2628

* readme: update release-3.0 description

* sql: add more info about load data (#2568)

* reference/performance: specify index name of primary key for USE_INDEX_MERGE (#2350)

Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>

* how-to: fix typos in placement rules doc (#2643)

* reference: update TiFlash tune doc (#2642)

* reference: update tiflash tune doc

* Update tune-performance.md

* Update reference/tiflash/tune-performance.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

Co-authored-by: Soup <maxiaoyu@pingcap.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>

* TiUP: document how to use TiUP to deploy TiDB v4.0 (#2635)

* Use of TiUP to deploy  TiDB of v4.0 version

* Use of TiUP to deploy  TiDB of v4.0 version

* Use of TiUP to deploy  TiDB of v4.0 version

* Use of TiUP to deploy  TiDB of v4.0 version

* refine content and format

* refine format

* Use of TiUP to deploy  TiDB of v4.0 version

* Use of TiUP to deploy  TiDB of v4.0 version

* Modify the file name to lowercase

Modify the file name to lowercase

* fix ci

* refine the image names in media for SEO reasons

* Use of TiUP to deploy  TiDB of v4.0 version

* Update how-to/deploy/orchestrated/tiup.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update how-to/deploy/orchestrated/tiup.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Use of TiUP to deploy  TiDB of v4.0 version

* refine content and format

* Update how-to/deploy/orchestrated/tiup.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* refine title

* Update how-to/deploy/orchestrated/tiup.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* add hyperlinks

* fix ci

* refine title

* remove an unfinished note

Co-authored-by: yikeke <yikeke@pingcap.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>

* tiflash: update tiflash deploy document (ansible -> tiup) (#2641)

* update tiflash doc for 4.0

* Update reference/tiflash/deploy.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update reference/tiflash/deploy.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update reference/tiflash/deploy.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update reference/tiflash/deploy.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update reference/tiflash/deploy.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update reference/tiflash/deploy.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update reference/tiflash/deploy.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update reference/tiflash/scale.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update reference/tiflash/deploy.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* minor edits to refine format

Co-authored-by: Soup <maxiaoyu@pingcap.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: yikeke <yikeke@pingcap.com>

* tiflash: update tiflash deploy document (ansible -> tiup) (#2641)

* update tiflash doc for 4.0

* Update reference/tiflash/deploy.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update reference/tiflash/deploy.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update reference/tiflash/deploy.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update reference/tiflash/deploy.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update reference/tiflash/deploy.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update reference/tiflash/deploy.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update reference/tiflash/deploy.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update reference/tiflash/scale.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* Update reference/tiflash/deploy.md

Co-Authored-By: Keke Yi <40977455+yikeke@users.noreply.github.com>

* minor edits to refine format

Co-authored-by: Soup <maxiaoyu@pingcap.com>
Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: yikeke <yikeke@pingcap.com>

* tiup: fix tiup deployment doc (#2651)

* Modify the details

* Modify the details

* reference/configuration: add description about TiDB txn-total-size-limit (#2456)

* reference/configuration: add tidb_row_format_version (#2605)

* Revert "Merge branch 'master' into pr/2650"

This reverts commit 2430b5c, reversing
changes made to 2ea8409.

Co-authored-by: Keke Yi <40977455+yikeke@users.noreply.github.com>
Co-authored-by: Lilian Lee <lilin@pingcap.com>
Co-authored-by: Lingyu Song <songlingyu@pingcap.com>
Co-authored-by: Kenan Yao <cauchy1992@gmail.com>
Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com>
Co-authored-by: xufei <xufei@pingcap.com>
Co-authored-by: Soup <maxiaoyu@pingcap.com>
Co-authored-by: lzs <43946384+superlzs0476@users.noreply.github.com>
Co-authored-by: yikeke <yikeke@pingcap.com>
Co-authored-by: Liangliang Gu <marsishandsome@gmail.com>
Co-authored-by: tiancaiamao <tiancaiamao@gmail.com>
Co-authored-by: lysu <sulifx@gmail.com>
@yikeke
Copy link
Contributor

yikeke commented Apr 14, 2020

/run-cherry-picker

1 similar comment
@yikeke
Copy link
Contributor

yikeke commented Apr 14, 2020

/run-cherry-picker

sre-bot pushed a commit to sre-bot/docs-cn that referenced this pull request Apr 14, 2020
@sre-bot
Copy link
Contributor

sre-bot commented Apr 14, 2020

cherry pick to release-4.0 in PR #2727

TomShawn pushed a commit that referenced this pull request Apr 14, 2020
…mit (#2456) (#2727)

Co-authored-by: tiancaiamao <tiancaiamao@gmail.com>
rleungx pushed a commit to rleungx/docs-cn that referenced this pull request May 22, 2020
@TomShawn TomShawn removed the requires-followup This PR requires a follow-up task after being merged. label Feb 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/small Changes of a small size. status/can-merge Indicates a PR has been approved by a committer. translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. v4.0 This PR/issue applies to TiDB v4.0.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants