Skip to content

v8.5.6: refresh TopSQL docs and screenshots (#22559)#22734

Merged
ti-chi-bot[bot] merged 2 commits intopingcap:feature/preview-v8.5.6from
ti-chi-bot:cherry-pick-22559-to-release-8.5
Apr 13, 2026
Merged

v8.5.6: refresh TopSQL docs and screenshots (#22559)#22734
ti-chi-bot[bot] merged 2 commits intopingcap:feature/preview-v8.5.6from
ti-chi-bot:cherry-pick-22559-to-release-8.5

Conversation

@ti-chi-bot
Copy link
Copy Markdown
Member

This is an automated cherry-pick of #22559

What is changed, added or deleted? (Required)

  • Refresh dashboard/top-sql.md to match the current pingcap/tidb-dashboard behavior for TopSQL.
  • Update the English TopSQL doc to cover Top 5/20/100, Order By CPU / Order By Network / Order By Logical IO, TiKV By Query / By Table / By DB / By Region, and the enable/disable behavior for TiKV network IO collection.
  • Re-shoot and replace the English TopSQL screenshots with the current English UI.
  • Add two new screenshots to explicitly show the Order By dropdown and the TopSQL settings drawer for enabling TiKV Network IO collection.
  • Update the Monitor TopSQL section in tidb-cloud/tidb-cloud-clinic.md so it describes multi-dimensional hotspot analysis instead of CPU-only analysis.

Which TiDB version(s) do your changes apply to? (Required)

  • master (the latest development version)
  • v9.0 (TiDB 9.0 versions)
  • v8.5 (TiDB 8.5 versions)
  • v8.1 (TiDB 8.1 versions)
  • v7.5 (TiDB 7.5 versions)
  • v7.1 (TiDB 7.1 versions)
  • v6.5 (TiDB 6.5 versions)
  • v6.1 (TiDB 6.1 versions)
  • v5.4 (TiDB 5.4 versions)

What is the related PR or file link(s)?

Do your changes match any of the following descriptions?

  • Delete files
  • Change aliases
  • Need modification after applied to another branch
  • Might cause conflicts after applied to another branch

@ti-chi-bot ti-chi-bot added lgtm needs-1-more-lgtm Indicates a PR needs 1 more LGTM. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. type/cherry-pick-for-release-8.5 This PR is cherry-picked to release-8.5 from a source PR. v9.0-beta.3 This PR/issue applies to TiDB v9.0-beta.3. labels Apr 13, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the Top SQL documentation to include the new multi-dimensional TiKV Network IO collection feature introduced in v8.5.6 and v9.0.0. This enhancement allows for monitoring network and logical I/O metrics with additional aggregation dimensions like Table, DB, and Region. Corresponding updates were made to TiDB Cloud Clinic and TiKV configuration documentation. Review feedback primarily suggested refinements to sentence structure, clarity, and terminology consistency to better align with the style guide.

Comment thread dashboard/top-sql.md
Comment thread dashboard/top-sql.md Outdated
Comment thread dashboard/top-sql.md
Comment thread dashboard/top-sql.md Outdated
Comment thread tidb-cloud/tidb-cloud-clinic.md Outdated
@qiancai qiancai assigned qiancai and unassigned yibin87 Apr 13, 2026
Comment thread dashboard/top-sql.md Outdated
Comment thread dashboard/top-sql.md Outdated
Comment thread tikv-configuration-file.md Outdated
Comment thread dashboard/top-sql.md Outdated
Comment thread dashboard/top-sql.md Outdated
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@qiancai qiancai changed the base branch from release-8.5 to feature/preview-v8.5.6 April 13, 2026 02:18
@qiancai qiancai changed the title refresh TopSQL docs and screenshots (#22559) v8.5.6: refresh TopSQL docs and screenshots (#22559) Apr 13, 2026
Copy link
Copy Markdown
Contributor

@yibin87 yibin87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ti-chi-bot
Copy link
Copy Markdown

ti-chi-bot Bot commented Apr 13, 2026

@yibin87: adding LGTM is restricted to approvers and reviewers in OWNERS files.

Details

In response to this:

LGTM

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@qiancai
Copy link
Copy Markdown
Collaborator

qiancai commented Apr 13, 2026

/approve

@ti-chi-bot
Copy link
Copy Markdown

ti-chi-bot Bot commented Apr 13, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: qiancai

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot ti-chi-bot Bot added the approved label Apr 13, 2026
@ti-chi-bot ti-chi-bot Bot merged commit dfc3c03 into pingcap:feature/preview-v8.5.6 Apr 13, 2026
12 checks passed
@ti-chi-bot ti-chi-bot Bot deleted the cherry-pick-22559-to-release-8.5 branch April 13, 2026 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved lgtm needs-1-more-lgtm Indicates a PR needs 1 more LGTM. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. type/cherry-pick-for-release-8.5 This PR is cherry-picked to release-8.5 from a source PR. v9.0-beta.3 This PR/issue applies to TiDB v9.0-beta.3.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants