-
Notifications
You must be signed in to change notification settings - Fork 204
[AURON #1926] ci: Remove opt-level=0 sed command to improve TPC-DS performance #1927
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
base: master
Are you sure you want to change the base?
Conversation
…-DS performance
|
Current CI TPC-DS test won't run, fixed by #1908 |
ShreyeshArangath
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we also need to change this to stay consistent?
https://github.com/apache/auron/blob/master/.github/workflows/build-arm-releases.yml#L104
The |
|
@cxzl25 PTAL |
If it is invalid, can it also be removed? |
Sure. |
ac1c1da to
f9b89cb
Compare
Which issue does this PR close?
Closes #1926
Rationale for this change
To improve TPC-DS performance in CI: reverting to
opt-level=1cuts TPC-DS runtime from 56 mins to 39 mins, reducing overall CI duration (compile time increase is negligible).What changes are included in this PR?
Are there any user-facing changes?
How was this patch tested?