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

OALP perftest tpch-2tiflash-50g& tpch-tiflash-50g result with total-avg-latency raise 15-20% #9586

Closed
stevenayu opened this issue Nov 5, 2024 · 4 comments · Fixed by #9587
Labels
affects-8.4 affects-8.5 This bug affects the 8.5.x(LTS) versions. severity/major

Comments

@stevenayu
Copy link

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

Deploy a tidb cluster with 3 TiDB (16c,48G) + 3 TiKV (16c,48G) + 2 TiFlash (16c,80G)
Compare the performance of v8.4.0 and v8.3.0 with TPCH benchmark

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

No performance regression

3. What did you see instead (Required)

total-avg-latency raise 15-20%.
image

4. What is your TiFlash version? (Required)

v8.4.0 (cea728d)

@wuhuizuo
Copy link
Contributor

wuhuizuo commented Nov 8, 2024

/close

it is not caused by codes, but the llvm toolchain in building env.

Copy link
Contributor

ti-chi-bot bot commented Nov 8, 2024

@wuhuizuo: Closing this issue.

In response to this:

/close

it is not caused by codes, but the llvm toolchain in building env.

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 kubernetes/test-infra repository.

@ti-chi-bot ti-chi-bot bot closed this as completed Nov 8, 2024
ti-chi-bot bot pushed a commit that referenced this issue Nov 8, 2024
close #9586

*: upgarde all docker images to use rocky linux 8

Signed-off-by: Lloyd-Pottiger <yan1579196623@gmail.com>

Co-authored-by: JaySon <tshent@qq.com>
@wuhuizuo
Copy link
Contributor

wuhuizuo commented Nov 8, 2024

/remove-type bug

@ti-chi-bot ti-chi-bot bot removed the type/bug The issue is confirmed as a bug. label Nov 8, 2024
ti-chi-bot bot pushed a commit that referenced this issue Nov 11, 2024
close #9586

*: upgarde all docker images to use rocky linux 8

Signed-off-by: Lloyd-Pottiger <yan1579196623@gmail.com>

Co-authored-by: Lloyd-Pottiger <yan1579196623@gmail.com>
Co-authored-by: Lloyd-Pottiger <60744015+Lloyd-Pottiger@users.noreply.github.com>
Co-authored-by: JaySon <tshent@qq.com>
@JaySon-Huang
Copy link
Contributor

JaySon-Huang commented Nov 14, 2024

Caused by when the CI pipeline base image migrate from centos 7 to rockylinux 8. And the LLVM toolchain has been changed during the migration.

Fixed by rebuilding the base image using the scripts in "/release-linux-llvm" and update the CI pipline

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-8.4 affects-8.5 This bug affects the 8.5.x(LTS) versions. severity/major
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants