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

Add description for tidb_distsql_scan_concurrency #5035

Merged
merged 7 commits into from
Dec 14, 2020

Conversation

glkappe
Copy link
Contributor

@glkappe glkappe commented Dec 7, 2020

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)

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

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

  • This PR is translated from:
  • Other reference link(s):

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

tidb_distsql_scan_concurrency Add change description 
 pingcap/tidb#19886
@TomShawn TomShawn assigned TomShawn and CharLotteiu and unassigned TomShawn Dec 7, 2020
@TomShawn TomShawn added needs-cherry-pick-4.0 size/small Changes of a small size. status/PTAL This PR is ready for reviewing. translation/doing This PR’s assignee is translating this PR. labels Dec 7, 2020
@@ -299,7 +299,9 @@ SET GLOBAL tidb_distsql_scan_concurrency = 10;
- 作用域:SESSION | GLOBAL
- 默认值:15
- 这个变量用来设置 scan 操作的并发度。
- `v4.0.6` 及其后面版本,该系统变量变更为 `tidb_executor_concurrency`
Copy link
Contributor

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 这个变量啊

Copy link
Contributor

Choose a reason for hiding this comment

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

我在 master 分支也没有搜到这个变量

Copy link
Contributor

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 中补充一下这个变量呢?

Copy link
Contributor

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 当中

@TomShawn TomShawn added status/require-change Needs the author to address comments. and removed status/PTAL This PR is ready for reviewing. labels Dec 8, 2020
@ti-srebot
Copy link
Contributor

@wshwsh12, @TomShawn, PTAL.

1 similar comment
@ti-srebot
Copy link
Contributor

@wshwsh12, @TomShawn, PTAL.

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

@wshwsh12, @TomShawn, PTAL.

@ti-srebot
Copy link
Contributor

@glkappe, please update your pull request.

@ti-srebot
Copy link
Contributor

@wshwsh12, @TomShawn, PTAL.

@ti-srebot
Copy link
Contributor

@glkappe, please update your pull request.

@ti-srebot
Copy link
Contributor

@, @, @, @, @, @, @wshwsh12, @TomShawn, PTAL.

ok

Co-authored-by: TomShawn <41534398+TomShawn@users.noreply.github.com>
@tiancaiamao
Copy link
Contributor

LGTM

@ti-srebot ti-srebot added the status/LGT1 Indicates that a PR has LGTM 1. label Dec 14, 2020
system-variables.md Outdated Show resolved Hide resolved
@TomShawn TomShawn changed the title tidb_distsql_scan_concurrency Add change description Add description for tidb_distsql_scan_concurrency Dec 14, 2020
@TomShawn TomShawn removed the status/require-change Needs the author to address comments. label Dec 14, 2020
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 removed the status/LGT1 Indicates that a PR has LGTM 1. label Dec 14, 2020
@ti-srebot ti-srebot added the status/LGT2 Indicates that a PR has LGTM 2. label Dec 14, 2020
@TomShawn TomShawn merged commit c5e8aad into pingcap:master Dec 14, 2020
ti-srebot pushed a commit to ti-srebot/docs-cn that referenced this pull request Dec 14, 2020
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot
Copy link
Contributor

cherry pick to release-4.0 in PR #5085

@CharLotteiu CharLotteiu 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 Dec 15, 2020
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/LGT2 Indicates that a PR has LGTM 2. 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