Conversation
|
Skipping CI for Draft Pull Request. |
ti-chi-bot
Bot
added
the
do-not-merge/work-in-progress
|
Important Review skippedToo many files! This PR contains 192 files, which is 42 over the limit of 150. To get a review, reduce the PR to 150 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (192)
You can disable this status message by setting the 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 |
ti-chi-bot
Bot
added
missing-translation-status
yahonda
changed the title
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…oted request kinds Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
yahonda
force-pushed
the
fix-yokyu-to-request-corpus-sweep
branch
from
September 8, 2026 07:40
303d3eb to
9a47477
Compare
…dentifiers Fixes 5 sites where a stray 目 (or 件目/13件目) was mistakenly attached to a backtick-quoted identifier, matching the established spurious-ordinal MT artifact class (see memory ja-spurious-banme-artifact): handle column mistranslated as handle-eye in 2 old release notes, a stray 目 inserted after tso_wait in performance-tuning-methods.md, and a TiCDC UPDATE/DELETE/ INSERT split-event backport notice garbled with spurious 目/件目/a stray '13' in release-7.1.6.md and release-7.5.2.md (release-6.5.10.md already had the correct rendering of the identical paragraph, used as the reference to fix the other two against). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
EN "highly concurrent" was mistranslated as 高度な同時 ("advanced
simultaneous") in 2 sites; the identical EN phrase is correctly rendered
elsewhere in the corpus as 同時実行性の高い (e.g. release-7.1.6.md and
release-7.5.3.md's copies of the same TiKV OOM release-note bullet), used
here as the reference to fix release-8.2.0.md's copy and
best-practices/high-concurrency-best-practices.md's intro sentence.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
sql-statements/sql-statement-explain-analyze.md: restored the missing は particle after `Batch_Point_Get` and reworded the noun+suru construction for 'read data' to the plain verb form for natural phrasing. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…oring doc ticdc/monitor-ticdc.md documents the literal panel/metric names shown on the actual TiCDC Grafana dashboards (both the new-architecture and classic-architecture monitoring boards). Nearly every bullet label across all 9 sections had been translated into Japanese, breaking the reader's ability to match a label against what they see in Grafana -- the same defect class documented for the grafana-*-dashboard.md files in PR pingcap#23763 (see memory ja-grafana-panel-names-english). Restored all metric/panel labels to literal English (matching the live Grafana panel/section names) while keeping the Japanese description text after each label unchanged. Also fixed 2 section headings and the panel-name references in prose that had drifted from their own anchor's English name (e.g. 概要/サーバー panel-name mentions not matching the Summary/Server heading english form). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…pace dynamic-config.md: aligned the TiFlash max_threads sentence with the established wording already used for the identical content in release-6.1.0.md (config item, not bare configuration), and removed 2 stray U+200B zero-width-space characters embedded mid-word (matching the known legacy MT artifact class). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…peer mistranslation - follower-read.md: restored a Grafana breadcrumb link (TiDB > KV Request > Read Req Traffic panel) to literal English, matching the panel-name defect class from PR pingcap#23763. - grafana-performance-overview-dashboard.md: restored all panel/metric labels across every section (Performance Overview, TiFlash, CDC) to literal English while keeping descriptions in Japanese; also fixed several scrambled/garbled sentences found in the same pass (QPS, avg by type, kv request by type bullets) and a duplicated Raft/Raft typo. - grafana-resource-control-dashboard.md: same treatment across all 3 metric sections. - glossary.md: fixed a same-file self-contradiction where the Region/Peer section heading and its inbound link used 仲間 (an informal, wrong register for 'peer') while the section's own body text already correctly used the established term ピア for the identical concept. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
6 tasks
yahonda
marked this pull request as ready for review
September 9, 2026 00:30
ti-chi-bot
Bot
removed
the
do-not-merge/work-in-progress
This was referenced Sep 9, 2026
Member
Author
yahonda
added a commit
to yahonda/docs
that referenced
this pull request
Sep 9, 2026
…shboard from this PR Both files fully overlapped with PR pingcap#23783/pingcap#23784 (the split halves of the 要求-to-リクエスト sweep, one of which had already fully restored grafana-performance-overview-dashboard.md's panel names). Since this branch was built from a pre-pingcap#23774 baseline, keeping these files here would revert that already-merged-in-spirit work and create a same-line merge conflict with pingcap#23783. The genuine new fixes from this PR (Raft apply/ready jargon, and the panel-name restoration for dashboard-monitoring.md that hadn't been done anywhere else) have been reapplied directly onto PR pingcap#23783's branch instead, so nothing is lost -- just moved to avoid duplicating/conflicting with already-in-flight work. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
yahonda
added a commit
to yahonda/docs
that referenced
this pull request
Sep 10, 2026
Split from PR pingcap#23774 into two halves to stay under CodeRabbit's 150-file review limit (originally 197 files). See PR pingcap#23774 for the full context and review history -- both halves together are this same corpus-wide term-unification plus the incidental mistranslation fixes found while reviewing it. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
yahonda
added a commit
to yahonda/docs
that referenced
this pull request
Sep 10, 2026
Split from PR pingcap#23774 into two halves to stay under CodeRabbit's 150-file review limit (originally 197 files). See PR pingcap#23774 for the full context and review history -- both halves together are this same corpus-wide term-unification plus the incidental mistranslation fixes found while reviewing it. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What is changed, added, or deleted? (Required)
Corpus-wide term-unification sweep: converts 要求 (yōkyū) to リクエスト (katakana "request") wherever the English source literally says "request" (noun or verb) — the established dominant form corpus-wide (リクエスト 1630 occurrences vs 要求 683 before this PR).
Every occurrence was individually checked against
upstream/release-8.5before converting. 要求 was intentionally LEFT UNCHANGED (36 remaining occurrences, across 25 files) wherever the EN source says something other than "request":system-variables.md,tidb-cloud/tidb-cloud-faq.md)tune-tikv-memory-performance.md,tidb-troubleshooting-map.md)tiup/tiup-cluster.md)Net result: 647 occurrences converted across 190 files (要求 683→36, リクエスト 1630→2277).
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?