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

dev: remove transaction kv count limit #1828

Merged
merged 6 commits into from
Oct 29, 2019
Merged

Conversation

jackysp
Copy link
Member

@jackysp jackysp commented Sep 6, 2019

Signed-off-by: Shuaipeng Yu jackysp@gmail.com

What is changed, added or deleted?

  1. Remove transaction kv count limit for dev.
  2. Remove the transaction limit and batch DML config in tidb-cluster, because they are hidden config.

What is the related PR or file link(s)?

pingcap/tidb#11141

Which version does your change affect?

dev

Signed-off-by: Shuaipeng Yu <jackysp@gmail.com>
Copy link
Contributor

@dcalvin dcalvin left a comment

Choose a reason for hiding this comment

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

LGTM

@lilin90 lilin90 added the translation/doing This PR’s assignee is translating this PR. label Sep 9, 2019
@jackysp
Copy link
Member Author

jackysp commented Sep 9, 2019

@tennix PTAL

@CaitinChen
Copy link
Contributor

@tennix PTAL

@CaitinChen CaitinChen requested review from tennix and removed request for CaitinChen September 23, 2019 07:07
@lilin90
Copy link
Member

lilin90 commented Oct 18, 2019

@DanielZhangQD PTAL

@@ -110,9 +110,6 @@ TiDB Operator 使用 Helm 部署和管理 TiDB 集群。通过 Helm 获取的配
| `tidb.txnLocalLatchesCapacity` | 事务内存锁的容量,Hash 对应的 slot 数,会自动向上调整为 2 的指数倍。每个 slot 占 32 Bytes 内存。当写入数据的范围比较广时(如导数据),设置过小会导致变慢,性能下降。<br>如果 TiDB Operator 版本 > v1.0.0-beta.3,请通过 `tidb.config` 配置:<br>`[txn-local-latches]`<br>`capacity = 10240000` | `10240000` |
| `tidb.tokenLimit` | TiDB 并发执行会话的限制<br>如果 TiDB Operator 版本 > v1.0.0-beta.3,请通过 `tidb.config` 配置:<br>`token-limit = 1000` | `1000` |
| `tidb.memQuotaQuery` | TiDB 查询的内存限额,默认 32GB<br>如果 TiDB Operator 版本 > v1.0.0-beta.3,请通过 `tidb.config` 配置:<br>`mem-quota-query = 34359738368` | `34359738368` |
| `tidb.txnEntryCountLimit` | 一个事务中条目的数目限制。如果使用 TiKV 作为存储,则条目表示键/值对。**警告**:不要将该值设置得太大,否则会对 TiKV 集群造成很大影响。请仔细调整此配置<br>如果 TiDB Operator 版本 > v1.0.0-beta.3,请通过 `tidb.config` 配置:<br>`[performance]`<br>`txn-entry-count-limit = 300000` | `300000` |
Copy link
Contributor

Choose a reason for hiding this comment

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

TiDB Operator supports different versions of TiKV, we cannot just remove the config, suggest to make it clear in which TiKV version that this parameter does not need to set.

Copy link
Member Author

Choose a reason for hiding this comment

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

I just change the docs for dev version, it means it will not work in the master branch. I think the Operator should follow the same rule.

Copy link
Contributor

@DanielZhangQD DanielZhangQD left a comment

Choose a reason for hiding this comment

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

Confirmed with @jackysp , the removed parameters in tidb-cluster.md should not be exposed to users to configure, so it's OK to remove them from the doc.
LGTM

@jackysp jackysp merged commit e62f6a9 into pingcap:master Oct 29, 2019
@TomShawn TomShawn added translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR. and removed translation/doing This PR’s assignee is translating this PR. labels Oct 29, 2019
@jackysp jackysp deleted the txn_kv_limit branch March 18, 2020 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
translation/done This PR has been translated from English into Chinese and updated to pingcap/docs-cn in a PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants