Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ui: show disk usage information for statement and slow query #777

Merged
merged 4 commits into from
Oct 28, 2020

Conversation

baurine
Copy link
Collaborator

@baurine baurine commented Oct 22, 2020

close #739

Screenshots:

image

image

These new columns seem to only exist in the new TiDB version, but I don't think we need to handle the compatibility problem before the dashboard has a standalone deployment mode, right?

Copy link
Member

@breezewish breezewish left a comment

Choose a reason for hiding this comment

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

Cool! As a reminder, please double-check that this field exists in latest v4.0 release. We only need to make sure that it is compatible with latest release-4.0 and latest master.

@baurine
Copy link
Collaborator Author

baurine commented Oct 23, 2020

Sure, will check it today.

@baurine
Copy link
Collaborator Author

baurine commented Oct 27, 2020

Checked that it works fine for the cluster of TiDB v4.0.7 and nightly.

WeCom20201027-112644@2x

WeCom20201027-112657@2x

WeCom20201027-113005@2x

Copy link
Member

@breezewish breezewish left a comment

Choose a reason for hiding this comment

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

Cool, should we show them in detail as well?

@baurine
Copy link
Collaborator Author

baurine commented Oct 27, 2020

Cool, should we show them in detail as well?

Let me add it.

@baurine
Copy link
Collaborator Author

baurine commented Oct 28, 2020

Add them to the detail page:

Statement:

image

SlowQuery:

image

@baurine baurine merged commit 594ccc1 into pingcap:master Oct 28, 2020
@baurine baurine deleted the disk-usage-info branch October 28, 2020 02:35
@breezewish
Copy link
Member

@SunRunAway Please have a try

breezewish added a commit that referenced this pull request Nov 26, 2020
* misc: Increase ulimit to 65535 for test env (#756)
* test: Fix frontend CI (#752)
* ui: fix dayjs i18n (#755)
* ui: handle error globally (#757)
* statement, slow_query: support all fields in list page (#749)
* ui: memorize expand/collapse full text in detail pages (#775)
* ui: break loop dependencies (#771)
* ui: fix browser compatibility check (#776)
* ui: Refine store location, add zoom and pan (#772)
* ui: show disk usage information for statement and slow query (#777)
* ui: use qps instead of ops (#786)
* statement: support export (#778)
*: Fix slow query and start_ts not working in some cases (#793)
* ui: fix errors doesn't display (#794)
* ui: fix the error message doesn't show correct (#799)
* slow_queries: support export (#792)
* ui: add MySqlFormatter to customize the sql formatter (#805)
*: fix query statement detail error cause by round (#806)
* ui: copy original content instead of formatted content for CopyLink (#802)
* add min height of topology canvas (#804)
* metrics: Support customize Prometheus address (#808)
* clusterinfo: Refine (#815)
* ui: Open statement and slow log in new tab (#816)
* ui: add more time field for slow query detail page (#810)
* slowlog: Improve descriptions (#817)
* build: add action to check release-version is changed for release branch
* Release v2020.11.26.1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support disk usage information in statement summary and slow query page
3 participants