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

The description of tidb_build_stats_concurrency in the documentation is incorrect #19364

Open
Rustin170506 opened this issue Nov 7, 2024 · 4 comments
Labels
severity/moderate type/bug The issue is confirmed as a bug.

Comments

@Rustin170506
Copy link
Member

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

Check https://docs.pingcap.com/zh/tidb/dev/statistics#tidb_build_stats_concurrency

2. What did you expect to see? (Required)

It should explain that it controls concurrency at the table or partition level.

3. What did you see instead (Required)

It says that this parameter controls the concurrency at the column level.

4. What is your TiDB version? (Required)

dev

@Rustin170506 Rustin170506 added type/bug The issue is confirmed as a bug. severity/minor severity/moderate and removed severity/minor labels Nov 7, 2024
@Rustin170506
Copy link
Member Author

The description of tidb_analyze_partition_concurrency is also incorrect. It only controls the write operations.

@Rustin170506
Copy link
Member Author

Also, it seems we do not use tidb_index_serial_scan_concurrency anymore.

@dveeden
Copy link
Contributor

dveeden commented Nov 7, 2024

Maybe this should have gone to https://github.com/pingcap/docs/ and/or https://github.com/pingcap/docs-cn/ That's also where you could create a PR if you want

@dveeden
Copy link
Contributor

dveeden commented Nov 7, 2024

cc @Oreoxmt @qiancai

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity/moderate type/bug The issue is confirmed as a bug.
Projects
None yet
Development

No branches or pull requests

2 participants