i18n(ja): unify Request Unit (RU) notation - #23702
Conversation
|
Skipping CI for Draft Pull Request. |
📝 WalkthroughWalkthroughThe pull request standardizes RU terminology across Japanese and English documentation. It also adds TiFlash scheduling guidance to the TiDB Cloud platform section. ChangesResource control documentation
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: 🔵 Low · up to The terminology updates are low risk, but TiDB Cloud readers may still be sent to an insecure, English self-managed documentation link instead of the applicable Japanese Cloud page. Correct the link before merge. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation 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 files. (7 skipped: 7 unsupported.) Full details: Description checkExplanation The description explains the terminology changes, affected files, target version, and AI involvement. The optional first-time contributor checklist is omitted, and the related-link fields are blank, but the required information is otherwise complete. ✨ 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 |
19fac0e to
99cefe7
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Team
Run ID: 0b78118f-f638-4c2d-a94d-b92785c5f959
📒 Files selected for processing (7)
dashboard/dashboard-resource-manager.mdgrafana-resource-control-dashboard.mdreleases/release-6.6.0.mdtidb-cloud/architecture-concepts.mdtidb-cloud/manage-serverless-spend-limit.mdtidb-cloud/select-cluster-tier.mdtidb-resource-control-ru-groups.md
Included review availability: Your plan provides up to 4 included reviews per hour; 2 remain after this review.
要求単位 and 要求ユニット were minority variant translations of "Request Unit" scattered across 7 files (13 occurrences), inconsistent with the corpus-dominant and glossary-defined term リクエストユニット (66 existing occurrences, including the glossary.md entry itself). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
99cefe7 to
79b0953
Compare
|
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. |
There was a problem hiding this comment.
Actionable comments posted: 1
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Team
Run ID: b28373bc-068a-4d41-afe3-af0c19f56901
📒 Files selected for processing (7)
dashboard/dashboard-resource-manager.mdgrafana-resource-control-dashboard.mdreleases/release-6.6.0.mdtidb-cloud/architecture-concepts.mdtidb-cloud/manage-serverless-spend-limit.mdtidb-cloud/select-cluster-tier.mdtidb-resource-control-ru-groups.md
Included review availability: Your plan provides up to 4 included reviews per hour; 0 remain after this review.
What is changed, added or deleted? (Required)
"Request Unit (RU)" had two minority variant translations scattered across the corpus — 要求単位 (9 occurrences) and 要求ユニット (3 occurrences) — inconsistent with the dominant, glossary-defined term リクエストユニット (66 existing occurrences, including the
glossary.mdentry itself:### リクエストユニット(RU) {#request-unit-ru}).Unified 13 occurrences across 7 files:
grafana-resource-control-dashboard.md(3 sites: RRU/WRU descriptions using 要求単位 and 要求ユニット, inconsistent with the RU/WRU-per-query lines in the same list which already correctly say リクエストユニット)dashboard/dashboard-resource-manager.md(2 sites)releases/release-6.6.0.md,tidb-cloud/architecture-concepts.md,tidb-cloud/manage-serverless-spend-limit.md,tidb-cloud/select-cluster-tier.md(2 sites),tidb-resource-control-ru-groups.md(2 sites)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