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

[GLUTEN-4670][VL] Fix split time metric for celeborn #4671

Merged
merged 1 commit into from
Feb 19, 2024

Conversation

kerwin-zk
Copy link
Contributor

What changes were proposed in this pull request?

Fixes: #4670

How was this patch tested?

CI

Copy link

github-actions bot commented Feb 7, 2024

#4670

Copy link

github-actions bot commented Feb 7, 2024

Run Gluten Clickhouse CI

@kerwin-zk
Copy link
Contributor Author

@ulysses-you cc

Copy link

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

1 similar comment
Copy link

Run Gluten Clickhouse CI

@ulysses-you ulysses-you merged commit 72960de into apache:main Feb 19, 2024
17 of 19 checks passed
@GlutenPerfBot
Copy link
Contributor

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query log/native_4671_time.csv log/native_master_02_18_2024_40486295d_time.csv difference percentage
q1 32.91 34.87 1.954 105.94%
q2 24.57 24.17 -0.401 98.37%
q3 38.06 38.03 -0.027 99.93%
q4 37.84 37.83 -0.009 99.98%
q5 70.51 72.33 1.820 102.58%
q6 7.04 8.29 1.250 117.75%
q7 83.02 84.22 1.195 101.44%
q8 83.71 85.03 1.322 101.58%
q9 124.59 120.47 -4.120 96.69%
q10 42.10 43.05 0.953 102.26%
q11 20.03 20.10 0.072 100.36%
q12 26.90 30.20 3.306 112.29%
q13 45.39 44.24 -1.150 97.47%
q14 16.34 19.94 3.603 122.06%
q15 27.12 27.56 0.441 101.63%
q16 13.76 14.18 0.416 103.02%
q17 100.53 102.14 1.609 101.60%
q18 148.59 146.95 -1.636 98.90%
q19 12.65 12.62 -0.022 99.82%
q20 28.62 27.97 -0.646 97.74%
q21 221.86 224.64 2.780 101.25%
q22 14.81 17.31 2.503 116.90%
total 1220.95 1236.16 15.214 101.25%

@GlutenPerfBot
Copy link
Contributor

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query log/native_master_02_19_2024_time.csv log/native_master_02_18_2024_40486295d_time.csv difference percentage
q1 33.43 34.87 1.432 104.28%
q2 24.73 24.17 -0.562 97.73%
q3 37.94 38.03 0.090 100.24%
q4 37.71 37.83 0.121 100.32%
q5 70.80 72.33 1.531 102.16%
q6 7.31 8.29 0.988 113.52%
q7 82.80 84.22 1.411 101.70%
q8 86.79 85.03 -1.758 97.97%
q9 121.45 120.47 -0.973 99.20%
q10 41.30 43.05 1.758 104.26%
q11 20.02 20.10 0.082 100.41%
q12 26.06 30.20 4.144 115.90%
q13 45.68 44.24 -1.437 96.85%
q14 15.10 19.94 4.842 132.07%
q15 27.77 27.56 -0.212 99.24%
q16 14.10 14.18 0.083 100.59%
q17 102.85 102.14 -0.712 99.31%
q18 149.16 146.95 -2.206 98.52%
q19 13.92 12.62 -1.298 90.68%
q20 26.87 27.97 1.106 104.12%
q21 226.51 224.64 -1.869 99.17%
q22 13.67 17.31 3.638 126.61%
total 1225.96 1236.16 10.199 100.83%

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.

[VL] Fix split time metric for celeborn
3 participants