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

Deprecate the concurrently-init-stats configuration #60206

Open
Rustin170506 opened this issue Mar 21, 2025 · 0 comments
Open

Deprecate the concurrently-init-stats configuration #60206

Rustin170506 opened this issue Mar 21, 2025 · 0 comments
Labels
component/statistics sig/planner SIG: Planner type/enhancement The issue or PR belongs to an enhancement.

Comments

@Rustin170506
Copy link
Member

Rustin170506 commented Mar 21, 2025

Enhancement

This configuration controls whether stats are initialized concurrently in non-lite init stats mode.

The current default value is true, as we believe it makes sense to initialize stats concurrently by default. The reasons are as follows:

  1. It only runs once when the TiDB node starts.
  2. For clusters with a large number of tables, single-threaded stats initialization is too slow. See Init stats failed: GC life time is shorter than transaction duration #60138
  3. There is no need to expose this implementation detail to users anymore.

So the proposal is to deprecate this configuration and initialize stats concurrently by default.

@Rustin170506 Rustin170506 added type/enhancement The issue or PR belongs to an enhancement. sig/planner SIG: Planner component/statistics labels Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/statistics sig/planner SIG: Planner type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Development

No branches or pull requests

1 participant