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

cop: add kv read wall time to print result #1248

Merged
merged 2 commits into from
Mar 27, 2024
Merged

Conversation

cfzjywxk
Copy link
Contributor

The kv read wall time which is the time consumed in tikv executors is not printted in the result.
It is helpful to check time consumed in the tikv next operations, for example processing column with larger values.

Signed-off-by: cfzjywxk <[email protected]>
@@ -663,6 +670,17 @@ func (td *TimeDetail) String() string {
return buf.String()
}

// Merge merges the time detail into itself.
func (td *TimeDetail) Merge(detail *TimeDetail) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Where is this used?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

To be used here https://github.com/pingcap/tidb/blob/master/pkg/distsql/select_result.go#L531 to record the TimeDetail information like the ScanDetail.

@cfzjywxk cfzjywxk requested a review from ekexium March 27, 2024 01:58
@cfzjywxk cfzjywxk merged commit 603dc7b into tikv:master Mar 27, 2024
10 checks passed
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.

3 participants