i18n(ja): fix dropped/duplicated particles in backported release-note bullets - #23708
Conversation
|
Skipping CI for Draft Pull Request. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Team Run ID: 📒 Files selected for processing (6)
Included review availability: Your plan provides up to 4 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe PR corrects Japanese wording in TiDB release notes for versions 6.5.10 through 8.1.1. It changes nine text lines across six release-note documents. No functional or structural changes are included. ChangesRelease note corrections
Estimated code review effort: 1 (Trivial) | ~3 minutes Merge Risk: ⚪ Minimal · up to This change corrects Japanese release-note wording only and introduces no product behavior changes or outstanding merge-readiness risk. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
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. Comment |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
cd024ad to
9deafab
Compare
[LGTM Timeline notifier]Timeline:
|
The bullet was missing the verb する connecting "log wo shutsuryoku" (output the log) to "mondai" (problem), reading as a broken noun compound instead of a relative clause. Same bug backported identically to 4 release files. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
… release-7.4.0.md EN: "when IndexLookup involves many table retrieval tasks". JA was missing the ni-particle marking IndexLookup as the container/subject of the clause. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
The bullet had a redundant "de" stacked right after "to" (jikkousuru to de), an MT artifact; the sibling occurrence of the same bug in release-8.2.0.md was already phrased correctly. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…convention EN: "Fix the overflow issue of the Longlong type in predicates". The possessive no-particle (predicate's Longlong type) subtly misstates this as the predicate's own type rather than a type used within predicates. Changed to niokeru, matching the sibling occurrence in release-8.2.0.md and the established convention used elsewhere in the corpus for "type error in X" bullets (e.g. release-4.0.11.md's "function niokeru duration type inference" bullet). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
9deafab to
35f2126
Compare
# Conflicts: # releases/release-6.5.10.md # releases/release-7.1.6.md # releases/release-7.4.0.md # releases/release-7.5.2.md # releases/release-7.5.3.md # releases/release-8.1.0.md # releases/release-8.1.1.md
|
Note GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer. |
What is changed, added or deleted? (Required)
Found while reading through release notes: several bullets (mostly backported bug-fix descriptions appearing identically across multiple release branches) had dropped or duplicated particles, or a particle choice that subtly misstated the relationship EN describes.
Fixed:
IndexLookupin release-7.4.0.md's cost-model bulletREMOVE PARTITIONINGbullet (sibling release-8.2.0.md already had it correct)Longlongoverflow-in-predicates bullet across 4 files (release-6.5.10.md, release-7.1.6.md, release-7.5.2.md, release-8.1.1.md), matching the already-correct sibling in release-8.2.0.md and the established corpus convention for "type error in X" bulletsEach fix was verified against the corresponding EN source in
release-8.5, using the issue/PR number as an anchor and cross-checking sibling occurrences of the same backported bug across release branches.Which TiDB version(s) do your changes apply to? (Required)
What is the related PR or file link(s)?
AI agent involvement
Do your changes match any of the following descriptions?
Summary by CodeRabbit
Longlongoverflow, synchronous statistics loading, andALTER TABLE ... REMOVE PARTITIONING.IndexLookup.