i18n(ja): restore literal English panel-name headings in Performance Overview dashboard docs - #23820
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 |
|
Warning Review limit reachedNext included review available in 25 minutes. View limit detailsLimit details: You’ve used all 4 included reviews currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: 📒 Files selected for processing (2)
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
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 (2)
Included review availability: Your plan provides up to 4 included reviews per hour; 2 remain after this review. 📝 WalkthroughWalkthroughBoth dashboard documents update the Connection Idle Duration description and rename the Execution Duration label to Execute Duration. ChangesDashboard metric terminology
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to The documentation updates consistently restore the intended dashboard metric names with no remaining merge-blocking risk. 🚥 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 |
…Overview dashboard docs Connection Idle Duration and 8 sibling headings are real Grafana panel titles (verified against pingcap/monitoring dashboard JSON) but were translated into Japanese, unlike their sibling bullet entries which were already kept literal. Also literalize the Connection Idle Duration panel-name reference in the intro sentence, which removes a tautological "duration...duration" repetition in Japanese as a side effect.
…ype-list wording Ten bullet labels under Duration/Parse-Compile-Execute Duration/ Storage-Async-Write-Store-Apply Duration/Append-Commit-Apply Log Duration reuse their section heading's name as the bullet's own subject, matching a real Grafana panel/series name, but were still translated into Japanese even after the heading itself was fixed. Also unify a stray Execution Duration to Execute Duration to match the real panel name, and fix a machine-translation word-order defect in three type-list bullets (QPS, KV/TSO Request OPS, avg by type) where SELECT/INSERT/UPDATE and Get/Prewrite/Commit tokens were scrambled out of their sentence context.
49380b0 to
5431b99
Compare
|
@coderabbitai full review |
|
What is changed, added or deleted? (Required)
Connection Idle Durationand 8 sibling section headings ingrafana-performance-overview-dashboard.mdanddashboard/dashboard-monitoring.mdare real Grafana panel titles (verified against theperformance_overview.jsondashboard definition in pingcap/monitoring), but were translated into Japanese instead of being kept literal, unlike their sibling bullet entries which were already literal English indashboard/dashboard-monitoring.md.This PR restores the literal English panel names in the affected headings:
It also literalizes the
Connection Idle Durationpanel-name reference in the intro sentence right below that heading in both files, which incidentally removes a tautological "duration ... duration" repetition in the Japanese sentence.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