-
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
Add description for tidb_distsql_scan_concurrency #5035
Conversation
tidb_distsql_scan_concurrency Add change description pingcap/tidb#19886
system-variables.md
Outdated
@@ -299,7 +299,9 @@ SET GLOBAL tidb_distsql_scan_concurrency = 10; | |||
- 作用域:SESSION | GLOBAL | |||
- 默认值:15 | |||
- 这个变量用来设置 scan 操作的并发度。 | |||
- `v4.0.6` 及其后面版本,该系统变量变更为 `tidb_executor_concurrency` |
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.
看起来4.0没有引入 tidb_executor_concurrency
这个变量啊
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.
我在 master 分支也没有搜到这个变量
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.
@wshwsh12 是在这个 PR 吗:pingcap/tidb#19886
是否需要在 docs-cn/system-variables.md at master · pingcap/docs-cn 中补充一下这个变量呢?
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.
@TomShawn pingcap/tidb#19886 引入的是 tidb_union_concurrency
变量,不是tidb_executor_concurrency
tidb_executor_concurrency
这个变量只在 master 上有,并没有引入到 4.0 当中
1 similar comment
@glkappe, please update your pull request. |
@glkappe, please update your pull request. |
ok Co-authored-by: TomShawn <[email protected]>
LGTM |
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
Signed-off-by: ti-srebot <[email protected]>
cherry pick to release-4.0 in PR #5085 |
tidb_distsql_scan_concurrency Add change description
pingcap/tidb#19886
First-time contributors' checklist
What is changed, added or deleted? (Required)
Which TiDB version(s) do your changes apply to? (Required)
What is the related PR or file link(s)?
Do your changes match any of the following descriptions?