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

sql plan changed and performance regression #50614

Closed
kennedy8312 opened this issue Jan 21, 2024 · 9 comments · Fixed by #51329
Closed

sql plan changed and performance regression #50614

kennedy8312 opened this issue Jan 21, 2024 · 9 comments · Fixed by #51329

Comments

@kennedy8312
Copy link

Bug Report

It seems to be a regression introduced by commit.

1. Minimal reproduce step (Required)

git checkout 86cd94a
make & run
mysql -u root -h 127.0.0.1 -P 4000 test < testcase.sql

sql file is not allowed to upload. Please change the file extension.
testcase_sql.log

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

Update_456 N/A 0 root time:4.85ms
testcase_result_bad.log

3. What did you see instead (Required)

Update_456 N/A 0 root time:27.2ms
testcase_result_good.log

4. What is your TiDB version? (Required)

The slowness happened on master as well. It started with #35669 (86cd94a).

@kennedy8312 kennedy8312 added the type/bug The issue is confirmed as a bug. label Jan 21, 2024
@kennedy8312
Copy link
Author

/type regression

@kennedy8312
Copy link
Author

Regression Analysis
PR caused this regression: #35669

@JasonWu0506
Copy link

/label affect-6.2, affect-6.5, affect-7.1, affect-7.5

@JasonWu0506
Copy link

/label affects-6.5

@JasonWu0506
Copy link

/label affects-7.1

@JasonWu0506
Copy link

/label affects-7.5

@JasonWu0506
Copy link

/label affects-6.2

Copy link

ti-chi-bot bot commented Jan 22, 2024

@JasonWu0506: The label(s) affects-6.2 cannot be applied. These labels are supported: fuzz/sqlancer, challenge-program, compatibility-breaker, first-time-contributor, contribution, good first issue, correctness, duplicate, proposal, security, ok-to-test, needs-ok-to-test, needs-more-info, needs-cherry-pick-release-5.4, needs-cherry-pick-release-6.1, needs-cherry-pick-release-6.5, needs-cherry-pick-release-7.1, needs-cherry-pick-release-7.5, needs-cherry-pick-release-7.6, affects-5.4, affects-6.1, affects-6.5, affects-7.1, affects-7.5, affects-7.6, may-affects-5.4, may-affects-6.1, may-affects-6.5, may-affects-7.1, may-affects-7.5, may-affects-7.6.

In response to this:

/label affects-6.2

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the ti-community-infra/tichi repository.

@ti-chi-bot ti-chi-bot bot added may-affects-5.4 This bug maybe affects 5.4.x versions. may-affects-6.1 labels Jan 25, 2024
@kennedy8312
Copy link
Author

/type performance

ti-chi-bot bot pushed a commit that referenced this issue Mar 28, 2024
ti-chi-bot bot pushed a commit that referenced this issue Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants