Skip to content

troubleshoot-tidb-cluster: fix TiDB error message text - #23621

Merged
ti-chi-bot[bot] merged 1 commit into
masterfrom
fix-en-information-schema-error-text
Sep 3, 2026
Merged

ti-chi-bot[bot] merged 1 commit into
masterfrom
fix-en-information-schema-error-text

Conversation

@yahonda

@yahonda yahonda commented Aug 27, 2026

Copy link
Copy Markdown
Member

What is changed, added or deleted? (Required)

Found while reviewing a Japanese translation fix (CodeRabbit review on #23618): troubleshoot-tidb-cluster.md says the error message is InformationSchema is out of date (one word), but this doesn't match the actual TiDB error text.

Verified against ErrInfoSchemaExpired in pingcap/tidb:

https://github.com/pingcap/tidb/blob/1f9469913409f0c6bf4a946c51948ebdfc2d2bb8/pkg/errno/errname.go#L991

ErrInfoSchemaExpired: mysql.Message("Information schema is out of date: schema failed to update in 1 lease, please make sure TiDB can connect to TiKV", nil),

The correct text is Information schema is out of date (with a space between "Information" and "schema"), matching 7 other occurrences of this string elsewhere in this same docs corpus (faq/sql-faq.md, several releases/*.md files, tidb-troubleshooting-map.md). Only this one file had the wrong spelling.

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

  • master

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

AI agent involvement

  • The changes in this PR were primarily made by an AI agent on behalf of the PR author.

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

@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 32 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 4 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Team

Run ID: c1408540-a60a-4665-b781-babb7143500d

📥 Commits

Reviewing files that changed from the base of the PR and between a49788b and 6e40413.

📒 Files selected for processing (1)
  • troubleshoot-tidb-cluster.md

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ti-chi-bot ti-chi-bot Bot added missing-translation-status This PR does not have translation status info. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Aug 27, 2026
The correct error text (per pkg/errno/errname.go's ErrInfoSchemaExpired
in pingcap/tidb) is "Information schema is out of date" with a space
between "Information" and "schema", not "InformationSchema" as one word.
@yahonda
yahonda force-pushed the fix-en-information-schema-error-text branch from 14bd773 to 6e40413 Compare September 2, 2026 02:44
@qiancai qiancai self-assigned this Sep 3, 2026
@ti-chi-bot ti-chi-bot Bot added the needs-1-more-lgtm Indicates a PR needs 1 more LGTM. label Sep 3, 2026
@ti-chi-bot

ti-chi-bot Bot commented Sep 3, 2026

Copy link
Copy Markdown

[LGTM Timeline notifier]

Timeline:

  • 2026-09-03 04:22:34.759755393 +0000 UTC m=+1339589.930849506: ☑️ agreed by qiancai.

@qiancai qiancai added translation/no-need No need to translate this PR. and removed needs-1-more-lgtm Indicates a PR needs 1 more LGTM. labels Sep 3, 2026
@ti-chi-bot ti-chi-bot Bot removed the missing-translation-status This PR does not have translation status info. label Sep 3, 2026
@qiancai qiancai added the needs-cherry-pick-master Should cherry pick this PR to master branch. label Sep 3, 2026
@qiancai

qiancai commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

/approve

@qiancai qiancai added the lgtm label Sep 3, 2026
@ti-chi-bot

ti-chi-bot Bot commented Sep 3, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: qiancai

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

The pull request process is described 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 the approved label Sep 3, 2026
@ti-chi-bot
ti-chi-bot Bot merged commit 854894d into master Sep 3, 2026
12 checks passed
@ti-chi-bot

Copy link
Copy Markdown
Member

In response to a cherrypick label: base branch (master) needs to differ from target branch (master).

@yahonda
yahonda deleted the fix-en-information-schema-error-text branch September 7, 2026 00:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved lgtm needs-cherry-pick-master Should cherry pick this PR to master branch. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. translation/no-need No need to translate this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants