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

select with limit cost exceed 40s but explain analyze only 100ms #20029

Closed
lzmhhh123 opened this issue Sep 16, 2020 · 5 comments · Fixed by #20030
Closed

select with limit cost exceed 40s but explain analyze only 100ms #20029

lzmhhh123 opened this issue Sep 16, 2020 · 5 comments · Fixed by #20030
Labels
component/tikv severity/major type/bug The issue is confirmed as a bug.

Comments

@lzmhhh123
Copy link
Contributor

lzmhhh123 commented Sep 16, 2020

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

load tpch_10 and create TiFlash replica for each table.

execute select * from lineitem limit 100

2. What did you expect to see? (Required)

SQL finished in a very short time.

3. What did you see instead (Required)

image

4. What is your TiDB version? (Required)

v4.0.5

5. W/A (Workaround)

use tikv instead tiflash.

6. Affected versions

v4.0.5

7. Fixed versions

v4.0.7

@sre-bot
Copy link
Contributor

sre-bot commented Sep 17, 2020

Integrity check:
component severity RCA symptom affect_version fix_version fields are empty

Please comment /info to get template

@pingcap pingcap deleted a comment from ti-srebot Sep 17, 2020
@seiya-annie
Copy link

/info

@ti-srebot
Copy link
Contributor

ti-srebot commented Sep 17, 2020

Please edit this comment to complete the following information

Not a bug

  1. Remove the 'type/bug' label
  2. Add notes to indicate why it is not a bug

Duplicate bug

  1. Add the 'type/duplicate' label
  2. Add the link to the original bug

Bug

Note: Make Sure that 'component', and 'severity' labels are added

1. Root Cause Analysis (RCA)

	// Because in rpc logic, context.Cancel() will be transferred to rpcContext.Cancel error. For rpcContext cancel,
	// we need to retry the request. But for context cancel active, for example, limitExec gets the required rows,
	// we shouldn't retry the request, it will go to backoff and hang in retry logic.

2. Symptom

see in bug body

3. All Trigger Conditions

see in the Minimal reproduce step (Required)

4. Workaround (optional)

use tikv instead tiflash.

5. Affected versions

[v4.0.5:4.0.5]

6. Fixed versions

v4.0.7

@lysu
Copy link
Contributor

lysu commented Sep 18, 2020

TiKV also became slow if user disables Superbatch(tikv-client.max-batch-size == 0)

it was introduced by 4.0.5 #18580

@seiya-annie
Copy link

add component/tikv label based on labels in PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/tikv severity/major type/bug The issue is confirmed as a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants