Skip to content

tidb: document analyze v1 removal#22655

Open
0xPoe wants to merge 1 commit intopingcap:masterfrom
0xPoe:poe-patch-analyze-v1-master-docs
Open

tidb: document analyze v1 removal#22655
0xPoe wants to merge 1 commit intopingcap:masterfrom
0xPoe:poe-patch-analyze-v1-master-docs

Conversation

@0xPoe
Copy link
Copy Markdown
Member

@0xPoe 0xPoe commented Mar 30, 2026

What is changed, added or deleted? (Required)

Document on master that Statistics Version 1 (tidb_analyze_version = 1) is no longer supported for new statistics collection. Update the statistics, system variable, and analyze status docs to describe the legacy read-only compatibility path and the migration and upgrade behavior.

Which TiDB version(s) do your changes apply to? (Required)

  • master (the latest development version)
  • v9.0 (TiDB 9.0 versions)
  • v8.5 (TiDB 8.5 versions)
  • v8.1 (TiDB 8.1 versions)
  • v7.5 (TiDB 7.5 versions)
  • v7.1 (TiDB 7.1 versions)
  • v6.5 (TiDB 6.5 versions)
  • v6.1 (TiDB 6.1 versions)
  • v5.4 (TiDB 5.4 versions)

What is the related PR or file 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

@ti-chi-bot
Copy link
Copy Markdown

ti-chi-bot bot commented Mar 30, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign lance6716 for approval. For more information see the Code Review Process.
Please ensure that each of them provides their approval before proceeding.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot ti-chi-bot bot added missing-translation-status This PR does not have translation status info. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Mar 30, 2026
@0xPoe 0xPoe changed the title statistics: document analyze v1 removal on master statistics: document analyze v1 removal Mar 30, 2026
@0xPoe 0xPoe changed the title statistics: document analyze v1 removal tidb: document analyze v1 removal Mar 30, 2026
@0xPoe 0xPoe changed the title tidb: document analyze v1 removal WIP: tidb: document analyze v1 removal Mar 30, 2026
@ti-chi-bot ti-chi-bot bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 30, 2026
@gemini-code-assist
Copy link
Copy Markdown
Contributor

gemini-code-assist bot commented Mar 30, 2026

Warning

Gemini is experiencing higher than usual traffic and was unable to create the review. Please try again in a few hours by commenting /gemini review.

@0xPoe 0xPoe force-pushed the poe-patch-analyze-v1-master-docs branch from 83f51f0 to 14da936 Compare March 31, 2026 08:29
@0xPoe
statistics: document analyze v1 removal
b541076 
fix: correct the job info

Signed-off-by: 0xPoe <techregister@pm.me>

fix

Signed-off-by: 0xPoe <techregister@pm.me>
@0xPoe 0xPoe force-pushed the poe-patch-analyze-v1-master-docs branch from 14da936 to b541076 Compare March 31, 2026 08:29
@0xPoe 0xPoe changed the title WIP: tidb: document analyze v1 removal tidb: document analyze v1 removal Mar 31, 2026
@ti-chi-bot ti-chi-bot bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Mar 31, 2026
0xPoe
0xPoe commented Mar 31, 2026
View reviewed changes
Copy link
Copy Markdown
Member Author

@0xPoe 0xPoe left a comment

Choose a reason for hiding this comment

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

🔢 Self-check (PR reviewed by myself and ready for feedback)

/cc @qw4990

@ti-chi-bot ti-chi-bot bot requested a review from qw4990 March 31, 2026 08:40
@ti-chi-bot
Copy link
Copy Markdown

ti-chi-bot bot commented Mar 31, 2026

@qw4990: adding LGTM is restricted to approvers and reviewers in OWNERS files.

Details

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@0xPoe 0xPoe requested a review from qiancai March 31, 2026 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@qw4990 qw4990 qw4990 approved these changes

@qiancai qiancai Awaiting requested review from qiancai

Assignees

No one assigned

Labels

missing-translation-status This PR does not have translation status info. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@0xPoe @qw4990