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

[CI] Improve CI to auto close stale patch #3634

Merged
merged 1 commit into from
Nov 9, 2023
Merged

Conversation

zhouyuan
Copy link
Contributor

@zhouyuan zhouyuan commented Nov 6, 2023

What changes were proposed in this pull request?

Auto close stale pull request after 45 days

How was this patch tested?

not tests required

Copy link

github-actions bot commented Nov 6, 2023

Thanks for opening a pull request!

Could you open an issue for this pull request on Github Issues?

https://github.com/oap-project/gluten/issues

Then could you also rename commit message and pull request title in the following format?

[GLUTEN-${ISSUES_ID}][COMPONENT]feat/fix: ${detailed message}

See also:

Copy link

github-actions bot commented Nov 6, 2023

Run Gluten Clickhouse CI

FelixYBW
FelixYBW previously approved these changes Nov 7, 2023
@FelixYBW
Copy link
Contributor

FelixYBW commented Nov 7, 2023

Thank you, @zhouyuan

Signed-off-by: Yuan Zhou <[email protected]>
Copy link

github-actions bot commented Nov 7, 2023

Run Gluten Clickhouse CI

@zhouyuan zhouyuan merged commit c7780fe into apache:main Nov 9, 2023
35 checks passed
@zhouyuan
Copy link
Contributor Author

zhouyuan commented Nov 9, 2023

Here's the example:
image

@GlutenPerfBot
Copy link
Contributor

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

query log/native_3634_time.csv log/native_master_11_08_2023_29b58997a_time.csv difference percentage
q1 33.81 34.11 0.301 100.89%
q2 25.06 25.13 0.073 100.29%
q3 39.13 38.62 -0.514 98.69%
q4 37.44 36.07 -1.365 96.35%
q5 71.08 71.84 0.760 101.07%
q6 7.59 7.59 -0.000 99.99%
q7 85.48 85.82 0.343 100.40%
q8 88.61 86.97 -1.636 98.15%
q9 126.22 121.86 -4.353 96.55%
q10 53.17 54.80 1.639 103.08%
q11 20.26 20.59 0.329 101.62%
q12 24.19 24.63 0.445 101.84%
q13 49.07 49.95 0.887 101.81%
q14 17.81 16.99 -0.819 95.40%
q15 30.88 30.97 0.095 100.31%
q16 16.29 16.51 0.226 101.39%
q17 100.24 102.40 2.155 102.15%
q18 149.24 148.28 -0.960 99.36%
q19 16.16 15.13 -1.027 93.64%
q20 29.39 29.97 0.578 101.97%
q21 223.81 222.20 -1.607 99.28%
q22 13.41 13.55 0.142 101.06%
total 1258.31 1254.01 -4.308 99.66%

zhouyuan added a commit that referenced this pull request Nov 16, 2023
follow up #3634
Increase the operations per run to 300

The default value is 30, which seems too small for gluten project

Signed-off-by: Yuan Zhou <[email protected]>
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.

4 participants