Skip to content
This repository was archived by the owner on Aug 28, 2024. It is now read-only.

Adding host based summary tables#3

Closed
aadant wants to merge 29 commits intomysql:masterfrom
aadant:master
Closed

Adding host based summary tables#3
aadant wants to merge 29 commits intomysql:masterfrom
aadant:master

Conversation

@aadant
Copy link
Contributor

@aadant aadant commented Mar 18, 2014

Based on existing user tables, adding host based tables.

aadant added 26 commits March 18, 2014 16:15
Mapped from user_summary_by_statement_type.sql
Mapped from user_summary_by_statement_latency.sql
Fixed the remaining users.
From user_summary_by_stages.sql
inspired from user_summary_by_file_io_type.sql
based on user_summary_by_file_io.sql
First draft : memory_by_host_by_current_bytes must be created as well.
fixed memory views
Fixed an ambiguous column.
updates the manual with host_* tables.
Would be nice to have a TOC (with links).
adding SOURCE ./views/p_s/waits_by_user_by_latency.sql
adding SOURCE ./views/p_s/waits_by_host_by_latency.sql
@aadant aadant closed this Mar 19, 2014
@aadant aadant reopened this Mar 19, 2014
Copy link
Member

Choose a reason for hiding this comment

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

Why was this (and following lines) removed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This item was duplicated, so I removed one.

README.md Outdated
Copy link
Member

Choose a reason for hiding this comment

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

This seems wrong, should it not be "select * from ..." in the example?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I cut and pasted, removing all * ... my mistake

@MarkLeith
Copy link
Member

This seems to be based off of the master branch, could you re-base this request off the development branch please, with the above problems fixed (all very minor clean up things).

You may need to close this request and open a new one, to base it off the development branch.

aadant added 3 commits March 19, 2014 10:34
removing the extra carriage return
Fixed all select *
removing extra lines
@aadant aadant closed this Mar 19, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants