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

Need some monitors for Multi-Statement #38930

Closed
jackysp opened this issue Nov 7, 2022 · 0 comments · Fixed by #38931
Closed

Need some monitors for Multi-Statement #38930

jackysp opened this issue Nov 7, 2022 · 0 comments · Fixed by #38931
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@jackysp
Copy link
Member

jackysp commented Nov 7, 2022

Enhancement

The multi-statement feature allows the clients to send SQLs in batch, which sometimes leads the users confused, such as, they will see the command per second is low and the QPS is high, and also they will see some SQLs will have a bad long tail latency, but they could not find the slow query in tidb-slow-query files. It is caused by the whole latency calculated as the last SQL one which is in a multi-statement.

It's better to add a metrcis or something else for multi-statemnt.

@jackysp jackysp added the type/enhancement The issue or PR belongs to an enhancement. label Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant