Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
add more time field for slow query detail page #810
add more time field for slow query detail page #810
Changes from 17 commits
fe1c1a9
1729986
043467d
4ce951f
6d022e0
9c424c7
c43ecdf
a3aad95
c005bb2
fea6b6b
10de098
cf9daa9
5607611
46b54a2
d90c6cf
71e6c8f
5e3d883
8f8d00b
dd40265
26bcd44
6626815
453c127
4190910
f38822e
a4e88e0
b08e954
388aecc
f66d54d
949f991
cb2decc
fd85096
cd56b4f
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is wait startTs? Is it the time of retrieving a StartTs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wait for retrieving a StartTs finish?
Since retrieving StartTs is asynchronous, when we need the
startTs
, we need wait for retrieving a StartTs finish first.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this a total time or P99 time or AVG time? As we know that there can be multiple Copr requests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The time model is below:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In 4.0 "cop requests might queue up" is not true, every requests will be processed in parallel. I recommend to not write this misleading information.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, this is copy from the documents.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like we need to update documents as well.. LOL