i18n(ja): add missing execution qualifier to plan cache proper noun - #23809
Conversation
|
[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 |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (6)
Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughThe change updates Japanese terminology for prepared and non-prepared execution plan caches in documentation headings, summaries, table-of-contents entries, and a benchmark reference. Link targets remain unchanged. ChangesPlan cache terminology
Estimated code review effort: 1 (Trivial) | ~2 minutes Suggested reviewers: Merge Risk: ⚪ Minimal · up to Japanese plan-cache documentation now consistently uses the execution-plan-cache terminology while preserving existing navigation and links. No current merge-readiness risk remains. 🚥 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 |
…config.md EN links to this page as "SQL Prepared Execution Plan Cache" (capitalized proper noun), while the adjacent "Non-prepared plan cache" link stays lowercase in EN -- so only the first link gains the qualifier, matching each site's own EN capitalization, consistent with pingcap#23809.
EN consistently names this feature "SQL (Non-)Prepared Execution Plan Cache" everywhere it appears as a proper noun (page titles, TOC entries, formal doc links), but the JA page titles themselves and 3 of 5 TOC files dropped the "execution" qualifier, while TOC.md, TOC-tidb-cloud-essential.md, and system-variable-reference.md already included it. Also translates sql-non-prepared-plan-cache.md's H1, which was left as literal English. Generic lowercase "prepared plan cache" mentions elsewhere in the corpus are untouched -- EN itself uses that shorter, casual form in many places, and JA correctly mirrors it there.
…config.md EN links to this page as "SQL Prepared Execution Plan Cache" (capitalized proper noun), while the adjacent "Non-prepared plan cache" link stays lowercase in EN -- so only the first link gains the qualifier, matching each site's own EN capitalization, consistent with pingcap#23809.
dc590a2 to
352623c
Compare
…config.md EN links to this page as "SQL Prepared Execution Plan Cache" (capitalized proper noun), while the adjacent "Non-prepared plan cache" link stays lowercase in EN -- so only the first link gains the qualifier, matching each site's own EN capitalization, consistent with pingcap#23809.
[LGTM Timeline notifier]Timeline:
|
…config.md EN links to this page as "SQL Prepared Execution Plan Cache" (capitalized proper noun), while the adjacent "Non-prepared plan cache" link stays lowercase in EN -- so only the first link gains the qualifier, matching each site's own EN capitalization, consistent with pingcap#23809.
What is changed, added or deleted? (Required)
EN consistently names this feature "SQL (Non-)Prepared Execution Plan Cache" everywhere it appears as a proper noun (page titles, TOC entries, formal doc links). The JA page titles themselves (
sql-prepared-plan-cache.md,sql-non-prepared-plan-cache.md) and 3 of 5 TOC files (TOC-tidb-cloud.md,TOC-tidb-cloud-premium.md,TOC-tidb-cloud-starter.md) dropped the "execution" qualifier (プリペアドプランキャッシュ), whileTOC.md,TOC-tidb-cloud-essential.md, andsystem-variable-reference.mdalready included it (プリペアド実行プランキャッシュ). Also translatessql-non-prepared-plan-cache.md's H1, which was left as literal English.Generic lowercase "prepared plan cache" mentions elsewhere in the corpus are intentionally untouched — EN itself uses that shorter, casual form in many places (verified line-by-line against
release-8.5), and JA correctly mirrors it there. Only sites where EN uses the capitalized proper-noun form were changed.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