Skip to content

i18n(ja): fix two translation defects in the v7.5.4 and v7.5.6 release notes - #24247

Open
yahonda wants to merge 1 commit into
pingcap:i18n-ja-release-8.5from
yahonda:fix-ja-release-7.5-deferred-lines
Open

yahonda wants to merge 1 commit into
pingcap:i18n-ja-release-8.5from
yahonda:fix-ja-release-7.5-deferred-lines

Conversation

@yahonda

@yahonda yahonda commented Oct 8, 2026 •

Copy link
Copy Markdown
Member

What is changed, added or deleted? (Required)

Fixes two translation defects in the Japanese v7.5.x release notes. These lines were found in the releases review but held back from #24154 because they are adjacent to lines changed in #24151. Both PRs are now merged.

  • releases/release-7.5.6.md: the Bit type was moved to the end of the sentence. It now appears with Enum and Set, matching EN: "When the parameter is of Enum, Bit, or Set type, the Conv() function is no longer pushed down to TiKV".
  • releases/release-7.5.4.md: "any" was missing in "between TiFlash and any PD". The fix adds 任意の.

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

Tips for choosing the affected version(s):

By default, CHOOSE MASTER ONLY so your changes will be applied to the next TiDB major or minor releases. If your PR involves a product feature behavior change or a compatibility change, CHOOSE THE AFFECTED RELEASE BRANCH(ES) AND MASTER.

For details, see tips for choosing the affected versions.

  • i18n-ja-release-8.5 (TiDB Japanese documentation for TiDB 8.5 versions)

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

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Clarified that a network partition between TiFlash and any PD can cause read-request timeouts.
    • Specified that Bit, like Enum and Set, is not eligible for Conv() pushdown.

…e notes

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ti-chi-bot

ti-chi-bot Bot commented Oct 8, 2026

Copy link
Copy Markdown

[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 tangenta 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/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Oct 8, 2026
@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: pingcap/docs/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 0ef9be76-34c1-4f2f-b0c9-b943095d36c6
📥 Commits

Reviewing files that changed from the base of the PR and between c070372 and bd69dfe.

📒 Files selected for processing (2)
  • releases/release-7.5.4.md
  • releases/release-7.5.6.md

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.


📝 Walkthrough

Walkthrough

Two release notes clarify the PD connection associated with a TiFlash timeout and add Bit to the Conv() parameter types that are not pushed down to TiKV.

Changes

TiFlash network-partition note

Layer / File(s) Summary
Clarify affected PD connection
releases/release-7.5.4.md
The note specifies that a network partition between TiFlash and any PD can cause read-request timeouts.

Conv pushdown note

Layer / File(s) Summary
Update excluded parameter types
releases/release-7.5.6.md
The note adds Bit to Enum and Set as parameter types that are not pushed down to TiKV.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to bd69d

The notes accurately describe the affected PD connections and the Bit pushdown limitation. No material PR-introduced risk remains before normal checks.

Architecture Summary

Architecture risk: 🔵 Low · up to bd69d

The change affects 1 system.

Changed systems: releases

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — releases (service) was modified; 2 changed files map to changed impact.

Before / after behavior

  • observed — Modified behavior in releases/release-7.5.4.md: The TiFlash network-partition issue description now identifies the affected connection as one between TiFlash and any PD; the prior description did not specify “any.”
  • observed — Modified behavior in releases/release-7.5.6.md: Conv() が TiKV にプッシュダウンされなくなる型として、従来の Enum と Set に Bit を追加しました。
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the two Japanese translation fixes and the affected v7.5.4 and v7.5.6 release notes.
Description check ✅ Passed The description explains both changes, identifies the target documentation branch, references related pull requests, and records AI involvement. It satisfies the required template sections.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

missing-translation-status This PR does not have translation status info. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant